Re: Remove the assert in Integer.valueOf()

[Ulf Zibis]

Am 23.04.2012 19:35, schrieb Rémi Forax:

Hi guys,
I've found a case where assert is harmful because it doesn't
play well with Hotspot inlining heuristic.
[...]
I think it's a good idea to comment this assert.

While you are there:
IntegerCache.cache/high/low are static final, so should be named _upper case_.

Another optimization:
    public static Integer valueOf(int i) {
        if (i >= 0) {
            if (i <= IntegerCache.HIGH)
                return IntegerCache.POS[i];
        } else {
            if (i >= IntegerCache.LOW)
                return IntegerCache.NEG[~i];
        }
        return new Integer(i);
    }


-Ulf