Hi Jonathan,

On 2017-12-06 21:58, Jonathan Bluett-Duncan wrote:
Hi Claes,

Looking at http://cr.openjdk.java.net/~redestad/8193128/open.00/src/java.base/share/classes/java/util/ImmutableCollections.java.cdiff.html <http://cr.openjdk.java.net/%7Eredestad/8193128/open.00/src/java.base/share/classes/java/util/ImmutableCollections.java.cdiff.html>, there are sections labelled --- 646,657 ---- and --- 834,845 ---- where lines like `Objects.requireNonNull(0 /* zero */);` are written. I believe that they were supposed to be either removed or made to be written like `Objects.requireNonNull(o /* lowercase o */)`. Is my belief/understanding correct?

nice catch! That was two fun little typos on my part (now why did we ever make 0 box to an Object...). Fixing in-place.

Thanks!

/Claes

Reply via email to