Am 29.09.2012 23:32, schrieb Kurchi Subhra Hazra:
On 9/29/12 2:23 PM, Ulf Zibis wrote:

Am 29.09.2012 22:52, schrieb Alan Bateman:

http://cr.openjdk.java.net/~alanb/8000269/webrev/index.html

Class.java :
You mean java.io.FilePermission.java?

Oops yes, thanks!


- in lines 702, 825 indentation should be 8.
- in line 826 I guess you wanted to move the curly brace to line 825
Curly Brace: I think that's the style we use when the method signature spreads 
across multiple lines.

To me this is an ugly break in style.

-Ulf

Reply via email to