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?

- 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.

- Kurchi

I've only reviewed the Sdiffs, so maybe there are more such cases, even in other classes.

-Ulf


Reply via email to