<div dir="ltr"><div id="m_1903007664727357079gmail-:1jo"><div id="m_1903007664727357079gmail-:1jk" aria-label="Message Body" role="textbox" aria-multiline="true" style="direction:ltr;min-height:376px" aria-controls=":2sr">Hi all,<div><br></div><div>I notice there is a field `Map<MethodInvocation, MethodInvocation> methods` </div><div>in the class `org.openjdk.jmh.generators.core.MethodGroup`.</div><div>It is used to record all the methods of one group and</div><div>the key and value of one entry of the map is always the same.</div><div>From the program's meaning, I think it is good to use a set instead of a map.</div><div>And this implementation confused the newbie, like me,</div><div>so that I needed more time to read the related code to confirm its meaning.</div><div><br></div><div>So I propose to change it from a map to a set.</div><div>I know the `TreeSet` is implemented by using a `TreeMap` internally,</div><div>but it is clearer about the program's meaning by using a set.</div><div><br></div><div>What do you think about it? Any ideas are appreciated.</div><div><br></div><div>Best Regards,</div><div>-- Guoxiong</div><div><br></div></div></div></div>