<div dir="ltr"><div>Just a couple of picky readability comments:     :)</div><div><br></div><div>  - IMO, the last line wrapped in an ifFalse: would allow removing three returns for one single return at the top of the method, allowing an easier hierarchical representation of the logic.</div>  - Comparing to 1000.0 instead of 1000 would provide a little better contextual readability.<div><br></div><div>Best,</div><div>  Chris</div><div><br><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Nov 9, 2019 at 1:13 PM <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">David T. Lewis uploaded a new version of System to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/System-dtl.1118.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox/System-dtl.1118.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: System-dtl.1118<br>
Author: dtl<br>
Time: 9 November 2019, 2:13:04.993358 pm<br>
UUID: 49b4ee31-b57f-4132-8cda-e13c2414f861<br>
Ancestors: System-mt.1117<br>
<br>
When reporting VM parameters, enable formatting in the case of a parameter value that is represented as a Float rather than an Integer. This may happen if a VM parameter is defined in units of milliseconds but internally measured to microsecond precision by the VM, in which case the value may be returned as a Float in order to retain full precision. SmalltalkImage>>formatVMParameter: handles the formatting for integer and non-integer values.<br>
<br>
=============== Diff against System-mt.1117 ===============<br>
<br>
Item was added:<br>
+ ----- Method: SmalltalkImage>>formatVMParameter: (in category 'vm statistics') -----<br>
+ formatVMParameter: aNumber<br>
+ <br>
+       aNumber isFloat ifTrue: [<br>
+               aNumber < 1000<br>
+                       ifTrue: [^ aNumber printShowingMaxDecimalPlaces: 5 ]<br>
+                       ifFalse: [^ aNumber rounded asStringWithCommas]].<br>
+       ^ aNumber asInteger asStringWithCommas.!<br>
<br>
Item was changed:<br>
  ----- Method: SmalltalkImage>>vmStatisticsReportOn: (in category 'vm statistics') -----<br>
(excessive size, no diff calculated)<br>
<br>
<br>
</blockquote></div>