[gwt-contrib] Typo in javadocs

2011-03-08 Thread T
Hi, This is just a tiny issue, so I don't want to involve issuetracking. I'm dealing with i18n translation, so I decided to generate properties from default locale. I've annotated my constants with @Generate as provided in:

Re: [gwt-contrib] Typo in javadocs

2011-03-08 Thread John Tamplin
On Tue, Mar 8, 2011 at 6:11 AM, T tomek.maty...@gmail.com wrote: This is just a tiny issue, so I don't want to involve issuetracking. I'm dealing with i18n translation, so I decided to generate properties from default locale. I've annotated my constants with @Generate as provided in:

[gwt-contrib] Re: Adds a cache (PersistenUnitCache) to store CompilationUnits (issue1375802)

2011-03-08 Thread jbrosenberg
http://gwt-code-reviews.appspot.com/1375802/diff/1/dev/core/src/com/google/gwt/dev/javac/PersistentUnitCache.java File dev/core/src/com/google/gwt/dev/javac/PersistentUnitCache.java (right):

[gwt-contrib] [google-web-toolkit] r9823 committed - Adding junit-stripped variation of testng, which used to embed junit

2011-03-08 Thread codesite-noreply
Revision: 9823 Author: fabb...@google.com Date: Tue Mar 8 08:59:35 2011 Log: Adding junit-stripped variation of testng, which used to embed junit http://code.google.com/p/google-web-toolkit/source/detail?r=9823 Added: /tools/lib/testng/README /tools/lib/testng/strip_junit.sh

[gwt-contrib] Re: Switching to the junit4 jars, although the @annotation stuff isn't (issue1374801)

2011-03-08 Thread Freeland Abbott
Pre-existing non-conforming, but yes, they should all use our jar. On Mon, Mar 7, 2011 at 4:24 PM, j...@google.com wrote: LGTM Why do some of our .classpath entries reference the JUnit jar directly, and others reference the Eclipse library JUNIT_CONTAINER? It seems likely that you need to

[gwt-contrib] [google-web-toolkit] r9824 committed - property fall back value evaluation scheme - enable fall back bindings...

2011-03-08 Thread codesite-noreply
Revision: 9824 Author: gwt.mirror...@gmail.com Date: Tue Mar 8 09:11:26 2011 Log: property fall back value evaluation scheme - enable fall back bindings. Review at http://gwt-code-reviews.appspot.com/1369807 http://code.google.com/p/google-web-toolkit/source/detail?r=9824 Added:

[gwt-contrib] Re: Switching to the junit4 jars, although the @annotation stuff isn't (issue1374801)

2011-03-08 Thread fabbott
http://gwt-code-reviews.appspot.com/1374801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Switching to the junit4 jars, although the @annotation stuff isn't (issue1374801)

2011-03-08 Thread jat
LGTM assuming you verified api-checker and cldr-import work with the new jars. http://gwt-code-reviews.appspot.com/1374801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] add ie9 user.agent with fallback binding to ie8. (in preparation for ie9 support). (issue1369808)

2011-03-08 Thread fabiomfv
Reviewers: rjrjr, jlabanca, jat, Description: add ie9 user.agent with fallback binding to ie8. (in preparation for ie9 support). Please review this at http://gwt-code-reviews.appspot.com/1369808/ Affected files: M user/src/com/google/gwt/user/UserAgent.gwt.xml Index:

[gwt-contrib] Re: add ie9 user.agent with fallback binding to ie8. (in preparation for ie9 support). (issue1369808)

2011-03-08 Thread John Tamplin
LGTM -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixing ActionCell and ButtonCell clicks outside of the Button element are ignored. (issue1371810)

2011-03-08 Thread pdr
On 2011/03/07 16:43:50, jlabanca wrote: adapted from http://gwt-code-reviews.appspot.com/1183801/show LGTM http://gwt-code-reviews.appspot.com/1371810/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] CLA response

2011-03-08 Thread Jens
Hi, I signed the individual CLA through the online form (I think it was last friday) and just wondering if there is any response from Google when its accepted? -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] gwt referencing a field not assigned - r9659

2011-03-08 Thread Deepak Khosla
Hi, Is there a way to tell if the checkin above http://code.google.com/p/google-web-toolkit/source/detail?r=9659 made it into the latest GWT 2.2? We have been having issues with our 'large' application using sencha GXT with GWT2.1.

[gwt-contrib] How to run a single test ?

2011-03-08 Thread Antoine DESSAIGNE
Hi, I've fixed an issue in gwt-user but I wasn't able to run the test I wrote. Running the test in Eclipse as a JUnit test fails and gives me a lot of com.google.gwt.core.ext.UnableToCompleteException. It tells me to look for the previous errors but I have so many of these

Re: [gwt-contrib] CLA response

2011-03-08 Thread Eric Ayers
No, there is no automated acknowledgement, but I just checked and can see that it got here OK. On Tue, Mar 8, 2011 at 4:53 AM, Jens jens.nehlme...@gmail.com wrote: Hi, I signed the individual CLA through the online form (I think it was last friday) and just wondering if there is any

[gwt-contrib] Re: gwt referencing a field not assigned - r9659

2011-03-08 Thread Sven Brunken
On 5 Mrz., 04:38, Deepak Khosla dkho...@x-iss.com wrote: Hi, Is there a way to tell if the checkin abovehttp://code.google.com/p/google-web-toolkit/source/detail?r=9659made it into the latest GWT 2.2? We have been having issues with our 'large' application using sencha GXT with

[gwt-contrib] Re: Adds a cache (PersistenUnitCache) to store CompilationUnits (issue1375802)

2011-03-08 Thread scottb
One other thing we should talk about is ditching all of the static-ness. The static-ness doesn't gel well with CompilationStateBuilder. CSB is usually a singleton, but you can instantiate an isolated CSB for testing which has a distinct cache that cannot be interfered with. It seems like CSB

[gwt-contrib] [google-web-toolkit] r9825 committed - Switching to the junit4 jars, although the @annotation stuff isn't goi...

2011-03-08 Thread codesite-noreply
Revision: 9825 Author: fabb...@google.com Date: Tue Mar 8 08:22:05 2011 Log: Switching to the junit4 jars, although the @annotation stuff isn't going to work. Also switching to the non-junit4 testng jars, to avoid classpath order confusion. Review at

[gwt-contrib] Re: add ie9 user.agent with fallback binding to ie8. (in preparation for ie9 support). (issue1369808)

2011-03-08 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1369808/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Adds a cache (PersistenUnitCache) to store CompilationUnits (issue1375802)

2011-03-08 Thread jbrosenberg
I agree with making PUC non-static. This reduces the need to synchronize on getting instance, etc. http://gwt-code-reviews.appspot.com/1375802/diff/5002/dev/core/src/com/google/gwt/dev/javac/PersistentUnitCache.java File dev/core/src/com/google/gwt/dev/javac/PersistentUnitCache.java (right):

[gwt-contrib] Fixed validation classifier, added clean plugin to prevent gwt-user from being deployed to GAE, ... (issue1377802)

2011-03-08 Thread drfibonacci
Reviewers: rchandia, Description: Fixed validation classifier, added clean plugin to prevent gwt-user from being deployed to GAE, upgraded to GAE 1.4.2 Please review this at http://gwt-code-reviews.appspot.com/1377802/ Affected files: M samples/expenses/pom.xml Index:

[gwt-contrib] Re: CompileModule / GwtAstBuilder (issue1373805)

2011-03-08 Thread zundel
LGTM, I have no substantive comments, go forth and code some more. I was asking myself if the stack approach might increase memory requirements, but the entire tree must already be in memory, so I stopped worrying about it.

[gwt-contrib] Re: CompileModule / GwtAstBuilder (issue1373805)

2011-03-08 Thread scottb
Okay.. so should I submit this? That way, future reviews will be vs. trunk. Maybe I should add a basic unit test to make sure that this at least doesn't crash, and maybe even that the output is the same as the web mode compiler for some test file.

[gwt-contrib] Re: CompileModule / GwtAstBuilder (issue1373805)

2011-03-08 Thread Eric Ayers
On Tue, Mar 8, 2011 at 9:36 PM, sco...@google.com wrote: Okay.. so should I submit this? That way, future reviews will be vs. trunk. Maybe I should add a basic unit test to make sure that this at least doesn't crash, and maybe even that the output is the same as the web mode compiler for

[gwt-contrib] Makes gadget linker shardable, derived from CrossSiteIframeLinker (issue1370808)

2011-03-08 Thread zundel
Reviewers: unnurg, http://gwt-code-reviews.appspot.com/1370808/diff/2001/gadgets/src/com/google/gwt/gadgets/linker/GadgetLinker.java File gadgets/src/com/google/gwt/gadgets/linker/GadgetLinker.java (right):

[gwt-contrib] PatternValidator matches the whole input string. (issue1378801)

2011-03-08 Thread nchalko
Reviewers: rchandia, Description: PatternValidator matches the whole input string. [JSR 303 TCK Result] 111 of 257 (43.19%) Pass with 19 Failures and 9 Errors. Please review this at http://gwt-code-reviews.appspot.com/1378801/ Affected files: M

[gwt-contrib] Handle custom constraint violation messages. (issue1379801)

2011-03-08 Thread nchalko
Reviewers: rchandia, Description: Handle custom constraint violation messages. [JSR 303 TCK Result] 110 of 257 (42.80%) Pass with 20 Failures and 9 Errors. Please review this at http://gwt-code-reviews.appspot.com/1379801/ Affected files: M

[gwt-contrib] Let the Key for Maps be of type Object. (issue1376803)

2011-03-08 Thread nchalko
Reviewers: rchandia, Description: Let the Key for Maps be of type Object. [JSR 303 TCK Result] 110 of 257 (42.80%) Pass with 20 Failures and 9 Errors. Please review this at http://gwt-code-reviews.appspot.com/1376803/ Affected files: M