Updated patch,

New output is very concise.

Compiling module com.example.MyApp
   Finding entry point classes
      [ERROR] Errors in 'com/example/TableFields.java'
         [ERROR] Line 26: The code for the static initializer is
exceeding the 65535 bytes limit
      [ERROR] Unable to find type 'com.example.EntryPoint'
         [ERROR] Hint: Previous compiler errors may have made this type
unavailable
         [ERROR] Hint: Check the inheritance chain from your module; it
may not be inheriting a required module or a module may not be adding
its source path entries properly



http://gwt-code-reviews.appspot.com/1416801/diff/4001/dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java
File
dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java
(right):

http://gwt-code-reviews.appspot.com/1416801/diff/4001/dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java#newcode250
dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java:250:
}
On 2011/04/13 20:45:08, scottb wrote:
These seems unnecessary.  Either CompilationState.getClassFileMap() or
getClassFileMapBySource() should give you a correct way to map type to
CompiledClass, then you just call cc.getUnit().

getClassFileMapBySource() works perfectly with apiDeps

http://gwt-code-reviews.appspot.com/1416801/diff/4001/dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java#newcode270
dev/core/src/com/google/gwt/dev/javac/CompilationProblemReporter.java:270:
}
I wrote some code to analyze the 2 sets of dependencies.  It isn't
needed.  In the end, it isn't this change that fixed it, but instead the
proper handling of inner classes.  The dependency I was looking at was:

-->EntryPoint
  -->Foo
    -->Bar$Baz

and since there is no unit name Bar$Baz, it just skipped it.

http://gwt-code-reviews.appspot.com/1416801/

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

Reply via email to