LGTM

I'm familiar with some of the dev mode stuff, so get a second opinion
from jat.


http://gwt-code-reviews.appspot.com/361801/diff/1/4
File dev/core/src/com/google/gwt/dev/DevModeBase.java (right):

http://gwt-code-reviews.appspot.com/361801/diff/1/4#newcode168
dev/core/src/com/google/gwt/dev/DevModeBase.java:168: public
ArgHandlerBlacklist() {
Is this constructor necessary?  Why wouldn't it exist by default?

http://gwt-code-reviews.appspot.com/361801/diff/1/4#newcode416
dev/core/src/com/google/gwt/dev/DevModeBase.java:416: public
ArgHandlerWhitelist() {
Necessary?

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

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

Reply via email to