Re: [gwt-contrib] [google-web-toolkit] r8503 committed - Fix datatype renderers to handle nulls properly for some primitive...

2010-08-09 Thread Pascal Patry
On Saturday, August 07, 2010 22:22:19 codesite-nore...@google.com wrote: Revision: 8503 Author: rj...@google.com Date: Sat Aug 7 16:11:36 2010 Log: Fix datatype renderers to handle nulls properly for some primitive types. We now expect renderers to produce for null, and parsers to return

[gwt-contrib] Re: r8484 committed - Fix support for null fields and Boolean type types in RequestFactory....

2010-08-09 Thread pjulien
Works great with both boolean and enumerations now On Aug 6, 6:05 pm, Ray Cromwell cromwell...@gmail.com wrote: I'm going to change it to treat undefined as null. In theory, I could change RecordJsoImpl to initialize all property fields to null or a default value,  according to JLS semantics,

[gwt-contrib] Deprecate the DeferredCommand API. (issue718802)

2010-08-09 Thread bobv
Reviewers: Ray Ryan, Description: Deprecate the DeferredCommand API. Patch by: bobv Review by: rjrjr Please review this at http://gwt-code-reviews.appspot.com/718802/show Affected files: M user/src/com/google/gwt/user/client/Command.java D

[gwt-contrib] Re: Deprecate the DeferredCommand API. (issue718802)

2010-08-09 Thread bobv
http://gwt-code-reviews.appspot.com/718802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] No GWT Developer Plugin for Chrome::Mac?

2010-08-09 Thread Brill
I find it extremely surprising that I can't use GWT with Chrome on my mac. I'm flabbergasted Does anyone know if the Google devs are planning on producing one? - Brill Pappin -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

Re: [gwt-contrib] No GWT Developer Plugin for Chrome::Mac?

2010-08-09 Thread John Tamplin
On Mon, Aug 9, 2010 at 10:44 AM, Brill br...@pappin.ca wrote: I find it extremely surprising that I can't use GWT with Chrome on my mac. I'm flabbergasted Does anyone know if the Google devs are planning on producing one?

[gwt-contrib] Fixing the msg key in ContactInfoForm so that the two buttons use different keys. (issue746801)

2010-08-09 Thread jlabanca
Reviewers: Dan Rice, Description: Fixing the msg key in ContactInfoForm so that the two buttons use different keys. Please review this at http://gwt-code-reviews.appspot.com/746801/show Affected files: M

[gwt-contrib] Re: Deprecate the DeferredCommand API. (issue718802)

2010-08-09 Thread BobV
I already have that. Do you want it in the same patch? -- Bob Vawter Google Web Toolkit Team -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Public: Simple implementation of ConstraintViolation, ConstraintDescriptor, Path and Node. (issue735802)

2010-08-09 Thread nchalko
http://gwt-code-reviews.appspot.com/735802/diff/1/10 File user/test/com/google/gwt/validation/client/ValidationClientGwtSuite.java (right): http://gwt-code-reviews.appspot.com/735802/diff/1/10#newcode23 user/test/com/google/gwt/validation/client/ValidationClientGwtSuite.java:23: * All

[gwt-contrib] [google-web-toolkit] r8504 committed - Refactor SelectionModel.SelectionEvent into its own class with associa...

2010-08-09 Thread codesite-noreply
Revision: 8504 Author: r...@google.com Date: Mon Aug 9 07:41:33 2010 Log: Refactor SelectionModel.SelectionEvent into its own class with associated Handler Review at http://gwt-code-reviews.appspot.com/698804 http://code.google.com/p/google-web-toolkit/source/detail?r=8504 Added:

[gwt-contrib] Re: Deprecate the DeferredCommand API. (issue718802)

2010-08-09 Thread bobv
Added callsite updates to the patch. http://gwt-code-reviews.appspot.com/718802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Patch from Gustav Trede (gustav.tr...@gmail.com) (issue716801)

2010-08-09 Thread scottb
Mostly LGTM. This looks like a nice cleanup and should remove a lot of potential for lock contention. I like that a lot of the code gets a lot simpler too. SerializabilityUtil.java is some particularly embarrassing code that I helped write. This patch should allow that code to be refactored

[gwt-contrib] [google-web-toolkit] r8505 committed - Public: Simple implementation of ConstraintViolation, ConstraintDescri...

2010-08-09 Thread codesite-noreply
Revision: 8505 Author: ncha...@google.com Date: Mon Aug 9 08:36:41 2010 Log: Public: Simple implementation of ConstraintViolation, ConstraintDescriptor, Path and Node. Review at http://gwt-code-reviews.appspot.com/735802 Review by: rj...@google.com

[gwt-contrib] [google-web-toolkit] r8506 committed - Fixing the msg key in ContactInfoForm so that the two buttons use diff...

2010-08-09 Thread codesite-noreply
Revision: 8506 Author: jlaba...@google.com Date: Mon Aug 9 08:45:34 2010 Log: Fixing the msg key in ContactInfoForm so that the two buttons use different keys. Review at http://gwt-code-reviews.appspot.com/746801 Review by: r...@google.com

[gwt-contrib] [google-web-toolkit] r8507 committed - Showcase: fix checkstyle errors and suppress deprecation warnings...

2010-08-09 Thread codesite-noreply
Revision: 8507 Author: r...@google.com Date: Mon Aug 9 08:47:15 2010 Log: Showcase: fix checkstyle errors and suppress deprecation warnings Review at http://gwt-code-reviews.appspot.com/747801 Review by: jlaba...@google.com http://code.google.com/p/google-web-toolkit/source/detail?r=8507

[gwt-contrib] Re: Public: Add default validators for the standard constraints. (issue735801)

2010-08-09 Thread scottb
Should Pattern.java have gone under user/super somewhere? The gwt-user eclipse project's checkstyle isn't happy about that file. http://gwt-code-reviews.appspot.com/735801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Public: Add default validators for the standard constraints. (issue735801)

2010-08-09 Thread nchalko
On 2010/08/09 19:34:56, scottb wrote: Should Pattern.java have gone under user/super somewhere? The gwt-user eclipse project's checkstyle isn't happy about that file. Yeah that file is excluded from that ant checkstyle. Let me know where would be good? The build / dir are still a bit of a

[gwt-contrib] Re: Comment on DomEventsAndMemoryLeaks in google-web-toolkit

2010-08-09 Thread codesite-noreply
Comment by j...@google.com: @bob.whiton: I'm assuming you mean something like this: B.addFooHandler(A); // drop reference to A This is just normal Java memory management -- nothing magic. So if B is still around (because it is, e.g., a widget still attached to the document), then A will

[gwt-contrib] Re: Faster edit-distance computation in JsFunctionClusterer (issue669801)

2010-08-09 Thread avassalotti
http://gwt-code-reviews.appspot.com/669801/diff/33001/34006 File dev/core/src/com/google/gwt/dev/jjs/impl/JsFunctionClusterer.java (right): http://gwt-code-reviews.appspot.com/669801/diff/33001/34006#newcode43 dev/core/src/com/google/gwt/dev/jjs/impl/JsFunctionClusterer.java:43:

[gwt-contrib] Re: Fixing the msg key in ContactInfoForm so that the two buttons use different keys. (issue746801)

2010-08-09 Thread jlabanca
comnmitted as r8506 http://gwt-code-reviews.appspot.com/746801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Deprecate the DeferredCommand API. (issue718802)

2010-08-09 Thread rjrjr
LGTM http://gwt-code-reviews.appspot.com/718802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] devmode classloader retention

2010-08-09 Thread Stephen Haberman
Hi, After reloading my app's module ~20 times, DevMode bogs down and has to be restarted. Watching it with a profiler, it seems the heap (500mb) is getting full and GC starts spiking the CPU. Permgen is fine. I took a heap snapshot with jmap, and in YourKit, it looks like the AppClassLoader is

[gwt-contrib] Better error message on garbage in text. (issue748801)

2010-08-09 Thread rjrjr
Reviewers: chenglee_google.com, Description: Better error message on garbage in text. Review by: cheng...@google.com Please review this at http://gwt-code-reviews.appspot.com/748801/show Affected files: M user/src/com/google/gwt/uibinder/rebind/XMLElement.java Index:

[gwt-contrib] Re: Replacing ListView with HasData and HasRows (issue745801)

2010-08-09 Thread rjrjr
LRDGTM http://gwt-code-reviews.appspot.com/745801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Refactor TypeOracle to prevent external mutations. (issue748802)

2010-08-09 Thread scottb
Reviewers: jat, Description: This change makes it so that TypeOracle can only be modified through an associated TypeOracleBuilder. This prevents Generators from changing the TypeOracle data model. Please review this at http://gwt-code-reviews.appspot.com/748802/show Affected files: M

Re: [gwt-contrib] devmode classloader retention

2010-08-09 Thread Scott Blum
Is the number of JRealClassType instances just going up and up each refresh? Can you figure out who is holding references to them (referrants)? On Mon, Aug 9, 2010 at 7:03 PM, Stephen Haberman step...@exigencecorp.comwrote: Hi, After reloading my app's module ~20 times, DevMode bogs down and

[gwt-contrib] Re: Hard coded History integration for the Scaffold app. This is step zero (issue717801)

2010-08-09 Thread rjrjr
Thanks, John. A question for you: should I get rid of the EventBus interface and instead just add addHandler() to HasHandlers? In theory it's a breaking change, but in practice are there really any HasHandler implementations other than our own?

[gwt-contrib] Re: Better error message on garbage in text. (issue748801)

2010-08-09 Thread rjrjr
http://gwt-code-reviews.appspot.com/748801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] RR: What if UiBinder could take a factory?

2010-08-09 Thread Ray Ryan
If you don't use GIN (you know, that really should be GIn), the rest of this note probably won't interest you. Would something like the following improve life for GIN users enough to be worth doing? Or is it just a hack? public interface UiBinderWithFactoryU, O, F extends UiBinderU, O { /**

[gwt-contrib] Re: Better error message on garbage in text. (issue748801)

2010-08-09 Thread rjrjr
http://gwt-code-reviews.appspot.com/748801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

Re: [gwt-contrib] devmode classloader retention

2010-08-09 Thread Stephen Haberman
Is the number of JRealClassType instances just going up and up each refresh? Yep. I didn't explicitly check until you asked, but comparing heap diffs before/after multiple refreshes, each refresh was adding ~1800 new JRealClassTypes that never went away. Can you figure out who is holding