http://gwt-code-reviews.appspot.com/886801/diff/13001/14006 File user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java (right):
http://gwt-code-reviews.appspot.com/886801/diff/13001/14006#newcode70 user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java:70: static class DefaultExceptionHandler implements ExceptionHandler { Make this a public, top-level type for end-users to extend in case the ExceptionHandler interface ever needs to grow. Also, add an @see in ExceptionHandler's javadoc to the default implementation. http://gwt-code-reviews.appspot.com/886801/diff/13001/14006#newcode76 user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java:76: private ExceptionHandler exceptionHandler; final http://gwt-code-reviews.appspot.com/886801/diff/13001/14006#newcode87 user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java:87: super(); Unnecessary. http://gwt-code-reviews.appspot.com/886801/diff/13001/14008 File user/src/com/google/gwt/requestfactory/shared/Receiver.java (right): http://gwt-code-reviews.appspot.com/886801/diff/13001/14008#newcode36 user/src/com/google/gwt/requestfactory/shared/Receiver.java:36: throw new RuntimeException(error.getExceptionType() + ": " That's going to look strange when the exception type is an empty string. http://gwt-code-reviews.appspot.com/886801/diff/13001/14009 File user/src/com/google/gwt/requestfactory/shared/ServerFailure.java (right): http://gwt-code-reviews.appspot.com/886801/diff/13001/14009#newcode34 user/src/com/google/gwt/requestfactory/shared/ServerFailure.java:34: String message, String exceptionType, String stackTraceString) { Formatting is off. http://gwt-code-reviews.appspot.com/886801/diff/13001/14009#newcode36 user/src/com/google/gwt/requestfactory/shared/ServerFailure.java:36: type = exceptionType; Inconsistent naming. http://gwt-code-reviews.appspot.com/886801/diff/13001/14010 File user/test/com/google/gwt/requestfactory/client/RequestFactoryTest.java (right): http://gwt-code-reviews.appspot.com/886801/diff/13001/14010#newcode450 user/test/com/google/gwt/requestfactory/client/RequestFactoryTest.java:450: assertEquals("Server Error", This value is inconsistent with a type name. http://gwt-code-reviews.appspot.com/886801/diff/13001/14010#newcode452 user/test/com/google/gwt/requestfactory/client/RequestFactoryTest.java:452: assertEquals("", error.getStackTraceString()); Use a test-specific ExceptionHandler that uses a non-empty string to ensure the field is being correctly transmitted. http://gwt-code-reviews.appspot.com/886801/diff/13001/14010#newcode453 user/test/com/google/gwt/requestfactory/client/RequestFactoryTest.java:453: finishTest(); You'll need to sync to head and change this to finishTestAndReset(). http://gwt-code-reviews.appspot.com/886801/diff/13001/14011 File user/test/com/google/gwt/requestfactory/client/impl/SimpleFooProxyProperties.java (right): http://gwt-code-reviews.appspot.com/886801/diff/13001/14011#newcode70 user/test/com/google/gwt/requestfactory/client/impl/SimpleFooProxyProperties.java:70: static final Property<Integer> pleaseCrash = new Property<Integer>("pleaseCrash", Integer.class); Is this actually used anywhere? The Property API has been made completely internal to the generated code. http://gwt-code-reviews.appspot.com/886801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
