RFR (XS): 8010222: 8007439 disabled inlining of cold accessor methods

John Rose john.r.rose at oracle.com
Mon Mar 18 12:32:51 PDT 2013


On Mar 18, 2013, at 11:31 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:

> http://cr.openjdk.java.net/~kvn/8010222/webrev/
> 
> Fixed typo which cause this.

It's a good fix.  Too bad about the typo.  — John

P.S. That sort of bug can crop up in an editor that doesn't do context-sensitive indenting.
I wonder if we should use a really basic style sniffer that would detect such "deceptive indents" such as:

. . .if (foo)
. . . .print("hello!");
. . . .return true;
or
. . .if (foo)  print("hello!");
. . . .return true;

Such things must exist somewhere.

(And, before anybody asks, I don't propose we start compensating for dumb text editors by requiring stricter or more uniform rules of style.  That would be so '80s.)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20130318/0eef6458/attachment.html 


More information about the hotspot-compiler-dev mailing list