Revision: 9664
Author: sco...@google.com
Date: Tue Feb 1 16:12:57 2011
Log: Fully process annotation types.
Binary annotations never become TypeDeclarations, so this short-circuit
code is unnecessarily preventing us from fully processing annotation
types. This is bad, because it's perfectly
SGTM.
Call it "gwt.js.inlinerRatio" maybe? There's wiki page somewhere that has
all the existing ones... there's probably a pattern that will emerge. (And
yours should be added.)
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
LGTM
On Tue, Feb 1, 2011 at 6:40 PM, wrote:
> Patch updated.
>
>
>
> http://gwt-code-reviews.appspot.com/1340802/diff/1/2
> File user/src/com/google/gwt/editor/client/EditorContext.java (right):
>
> http://gwt-code-reviews.appspot.com/1340802/diff/1/2#newcode114
> user/src/com/google/gwt/editor/
Revision: 9663
Author: to...@google.com
Date: Tue Feb 1 13:54:46 2011
Log: Speed up DevMode by getting rid of excessive JS to Java calls.
Review at http://gwt-code-reviews.appspot.com/1337801
Review by: zun...@google.com
http://code.google.com/p/google-web-toolkit/source/detail?r=9663
Modified
Patch updated.
http://gwt-code-reviews.appspot.com/1340802/diff/1/2
File user/src/com/google/gwt/editor/client/EditorContext.java (right):
http://gwt-code-reviews.appspot.com/1340802/diff/1/2#newcode114
user/src/com/google/gwt/editor/client/EditorContext.java:114: * Traverse
a editor created by
http://gwt-code-reviews.appspot.com/1340802/diff/1/4
File user/src/com/google/gwt/editor/client/EditorVisitor.java (right):
http://gwt-code-reviews.appspot.com/1340802/diff/1/4#newcode27
user/src/com/google/gwt/editor/client/EditorVisitor.java:27: public
void endVisit(EditorContext ctx) {
Given
http://gwt-code-reviews.appspot.com/1236801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: scottb,
Description:
Adds option to override JsInliner's max complexity ratio heuristic with
JVM property 'gwt.jsinlinerratio'. The default is 5.0, smaller values
tend to reduce code size. Empirically, 1.2 seems to be a good value.
Please review this at http://gwt-code-reviews.appspo
Revision: 9662
Author: gwt.mirror...@gmail.com
Date: Tue Feb 1 14:42:07 2011
Log: Update plural rules for Gaelic and Welsh.
Patch by: jat
Review by: pdr
http://code.google.com/p/google-web-toolkit/source/detail?r=9662
Modified:
/trunk/user/src/com/google/gwt/i18n/client/impl/plurals/DefaultRu
LGTM
http://gwt-code-reviews.appspot.com/1340801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1340801/diff/1/2
File dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
(right):
http://gwt-code-reviews.appspot.com/1340801/diff/1/2#newcode453
dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java:453:
public void processType(TypeDeclarat
committed as r9660
http://gwt-code-reviews.appspot.com/1310804/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/1340801/diff/1/2
File dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java
(right):
http://gwt-code-reviews.appspot.com/1340801/diff/1/2#newcode453
dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaAST.java:453:
public void processType(TypeDeclarat
Revision: 9661
Author: zun...@google.com
Date: Tue Feb 1 08:37:27 2011
Log: Refactors DefaultFilters and DefaultFiltersTest to accept a filter
that will return .class files.
Review at http://gwt-code-reviews.appspot.com/1336801
http://code.google.com/p/google-web-toolkit/source/detail?r=9661
On Tue, Feb 1, 2011 at 2:07 PM, wrote:
> If I understand correctly, the answer to
> http://code.google.com/p/google-web-toolkit/issues/detail?id=5892 would
> be to do it in user-code? (or at least it would make it possible)
That kind of problem is exactly the reason that the visitor is being
mad
If I understand correctly, the answer to
http://code.google.com/p/google-web-toolkit/issues/detail?id=5892 would
be to do it in user-code? (or at least it would make it possible)
Something like:
// assuming a 'final RequestContext context'
driver.accept(new EditorVisitor() {
@Override
public
http://gwt-code-reviews.appspot.com/1340802/diff/1/2
File user/src/com/google/gwt/editor/client/EditorContext.java (right):
http://gwt-code-reviews.appspot.com/1340802/diff/1/2#newcode114
user/src/com/google/gwt/editor/client/EditorContext.java:114: * Traverse
a editor created by
an editor
http
Revision: 9660
Author: jlaba...@google.com
Date: Tue Feb 1 07:22:51 2011
Log: Handling errors more from user code more gracefully in
HasDataPresenter. If Cells, Cell Widgets, or the SelectionModel throw
exceptions during the rendering loop, we no longer lock the presenter's
rendering loop i
Reviewers: rice,
Description:
Remove unnecessary "script src=..." from MediaTest.gwt.xml to avoid
virus scanner false positives
A user reported that Sophos antivirus was flagging MediaTest.gwt.xml
(see: http://savmac7-20.p.link.sophos.com/t/en/Mal/Badsrc-D). The script
src=... line isn't necessa
LGTM
http://gwt-code-reviews.appspot.com/1310804/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Rietveld makes this look like a bigger patch than it is. It really just
adds some try/catch blocks around potentional calls to User code.
http://gwt-code-reviews.appspot.com/1310804/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: sbrubaker,
Description:
Handling errors more from user code more gracefully in HasDataPresenter.
If Cells, Cell Widgets, or the SelectionModel throw exceptions during
the rendering loop, we no longer lock the presenter's rendering loop
indefinitely.
Please review this at http://gwt-c
Reviewers: rjrjr,
Description:
Overhaul Editor framework traversal logic to use a visitor pattern.
Add a base EditorDriver interface, with isDirty().
Patch by: bobv
Review by: rjrjr
Please review this at http://gwt-code-reviews.appspot.com/1340802/show
Affected files:
A user/src/com/google/g
23 matches
Mail list logo