On 6/6/19 12:35 PM, Ivan Gerasimov wrote: > Could you please check it if it looks better now? > > http://cr.openjdk.java.net/~igerasim/8225397/00/webrev/
OK, it's just a hint, fair enough. Patch is OK. For an extra gold star (:-) you could consider adding a comment like "avoid overflow in the case of a humongous numBits". There's no need for another review if you do so. -- Andrew Haley Java Platform Lead Engineer Red Hat UK Ltd. <https://www.redhat.com> https://keybase.io/andrewhaley EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
