<p dir="ltr">I initially read the comment as meaning it memoizes the name of the property (and not the value) but re-reading it again, I think it's fine.</p>
<p dir="ltr">sent from my phone</p>
<div class="gmail_quote">On Apr 6, 2015 9:53 PM, "Igor Veresov" <<a href="mailto:igor.veresov@oracle.com">igor.veresov@oracle.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Vitaly,<div><br></div><div>Thanks, others already noted the typo.</div><div>The comment means that we memoize the result of bruteforcing of the name of the property. I’m quite sure what is the confusion? What would you like it to say?</div><div><br></div><div>igor </div><div><br><div><blockquote type="cite"><div>On Apr 6, 2015, at 5:13 PM, Vitaly Davidovich <<a href="mailto:vitalyd@gmail.com" target="_blank">vitalyd@gmail.com</a>> wrote:</div><br><div><p dir="ltr">Hi Igor,</p><p dir="ltr">// One the first visit determine the name of the l2 cache line size property and memoize it</p><p dir="ltr">Typo - should be "On the first ..." I presume.</p><p dir="ltr">Also, that code doesn't just memoize the property name, it also appears to actually probe for and set the value - worthwhile to update the comment?</p><p dir="ltr">sent from my phone</p>
<div class="gmail_quote">On Apr 6, 2015 8:00 PM, "Igor Veresov" <<a href="mailto:igor.veresov@oracle.com" target="_blank">igor.veresov@oracle.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks, Vladimir!<br>
<br>
igor<br>
<br>
> On Apr 6, 2015, at 3:59 PM, Vladimir Kozlov <<a href="mailto:vladimir.kozlov@oracle.com" target="_blank">vladimir.kozlov@oracle.com</a>> wrote:<br>
><br>
> Good.<br>
><br>
> Thanks,<br>
> Vladimir<br>
><br>
> On 4/6/15 3:46 PM, Igor Veresov wrote:<br>
>> The L2 data cache line size property can be name either "l2-cache-line-size” or “l2-dcache-line-size”. We have to try them both.<br>
>><br>
>> Webrev: <a href="http://cr.openjdk.java.net/~iveresov/8076968/webrev.00/" target="_blank">http://cr.openjdk.java.net/~iveresov/8076968/webrev.00/</a><br>
>><br>
>> Thanks,<br>
>> igor<br>
>><br>
<br>
</blockquote></div>
</div></blockquote></div><br></div></div></blockquote></div>