Thanks.

http://gwt-code-reviews.appspot.com/748802/diff/1/3
File dev/core/src/com/google/gwt/core/ext/typeinfo/JAbstractMethod.java
(right):

http://gwt-code-reviews.appspot.com/748802/diff/1/3#newcode270
dev/core/src/com/google/gwt/core/ext/typeinfo/JAbstractMethod.java:270:
realParameterNames =
getEnclosingType().getOracle().getJavaSourceParser().getArguments(
Well, that was what autoformat did, but I'll revert.

http://gwt-code-reviews.appspot.com/748802/diff/1/6
File dev/core/src/com/google/gwt/core/ext/typeinfo/JArrayType.java
(right):

http://gwt-code-reviews.appspot.com/748802/diff/1/6#newcode199
dev/core/src/com/google/gwt/core/ext/typeinfo/JArrayType.java:199:
LOL, these spaces actually sorted from line 194, I think.  Will fix.

http://gwt-code-reviews.appspot.com/748802/diff/1/17
File
dev/core/src/com/google/gwt/core/ext/typeinfo/JParameterizedType.java
(right):

http://gwt-code-reviews.appspot.com/748802/diff/1/17#newcode102
dev/core/src/com/google/gwt/core/ext/typeinfo/JParameterizedType.java:102:
assert (this.typeArgs.indexOf(null) == -1) : "Unresolved typeArg
creating JParameterizedType from "
On 2010/08/12 20:14:35, jat wrote:
The previous formatting seems better -- this breaks awkwardly and
exceeds 80
characters.

Done.

http://gwt-code-reviews.appspot.com/748802/diff/1/20
File dev/core/src/com/google/gwt/core/ext/typeinfo/JType.java (right):

http://gwt-code-reviews.appspot.com/748802/diff/1/20#newcode23
dev/core/src/com/google/gwt/core/ext/typeinfo/JType.java:23: JType() {
Prevents user subclassing.

http://gwt-code-reviews.appspot.com/748802/diff/1/26
File dev/core/src/com/google/gwt/dev/javac/TypeOracleMediator.java
(right):

http://gwt-code-reviews.appspot.com/748802/diff/1/26#newcode204
dev/core/src/com/google/gwt/dev/javac/TypeOracleMediator.java:204: *
That's what autoformat does.

http://gwt-code-reviews.appspot.com/748802/show

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to