The point of removing the calls was correctness (for future subclasses),
not performance.
Dan
On Thu, May 14, 2009 at 10:57 AM, John Tamplin j...@google.com wrote:
On Thu, May 14, 2009 at 10:31 AM, Lex Spoon sp...@google.com wrote:
Any compile error about overriding a final method
I see around 2x for the story generation phase. It's hard to measure at
the moment because I haven't been able to get a working before build so I
am estimating based on some numbers jotted down earlier.
Dan
On Tue, May 26, 2009 at 2:27 PM, Bruce Johnson br...@google.com wrote:
Any numbers
I think simply because they were unreferenced. I'll restore them.
Dan
On Mon, Jun 1, 2009 at 12:29 PM, Scott Blum sco...@google.com wrote:
Dan, can you shed some light? I think you removed them in r5410.
On Sun, May 31, 2009 at 9:40 PM, Amir amirkash...@gmail.com wrote:
It looks like
I have verified that the xml files for a single-permutation Showcase
build are identical modulo whitespace to a set of golden xml files.
I would be happy to run the dashboard on a larger compile -- how do I
run it?
I already submitted a CL, so I will make this changes in a follow-up CL.
Dan
? Probably just some super naive testing on my part. :)
On Thu, Jun 4, 2009 at 12:32 PM, Daniel Rice (דניאל רייס)
r...@google.com wrote:
The speedup was measured over 3 non-JProfiler runs, invoked within
Eclipse. It's certainly possible that some of the difference is
normal runtime
In this case, the code is the GWT compiler itself, so it doesn't get
translated ot JavaScript.
Dan
On Fri, Jun 5, 2009 at 10:15 AM, TazmanianDdamon.lun...@gmail.com wrote:
There is an additional benefit to this optimization that goes beyond
speeding up the compiler. It should produce
(דניאל רייס)r...@google.com
wrote:
It would not be easy to force all of these lists to have some
particular concrete implementation (FinalArrayList) although some
could. I can repeat the analysis with a larger number of runs just to
be sure, but the speedup was seen in the context
Hi Bruno -
I've uploaded a new patch that's more along these lines. Hopefully it
will do what you are looking for. Thanks,
Dan
On Tue, Jun 30, 2009 at 4:15 AM, bruno bruno.marches...@gmail.com wrote:
Hi Bod and Dan,
Very happy to see that this issue is on its way to be solved :)
Please ignore this, I now see that there is not a problem with the
existing code.
Dan
On Thu, Jul 16, 2009 at 2:20 PM, r...@google.com wrote:
Reviewers: robertvawter_google.com,
Please review this at http://gwt-code-reviews.appspot.com/47819
Affected files:
I'll give it a shot.
Dan
On Thu, Jul 16, 2009 at 4:17 PM, Ray Ryan rj...@google.com wrote:
Dan, can you add the test that would have caught that for you?
On Thu, Jul 16, 2009 at 10:47 AM, Daniel Rice (דניאל רייס)
r...@google.com wrote:
Sorry, fixed.
Dan
On Thu, Jul 16, 2009 at 1
Hi all -
I've been working on a patch to improve support for RPC of
persistence-enhanced objects that would replace the one I recently submitted
as trunk revision 5672. I'd like to give those of you who are interested in
the interaction between RPC and persistence a chance to help me validate
setXXX() method invocations
rather
than by directly setting field values
Is this introducing a requirement that all serializable fields have
setters? Or will direct field access be used where a setter can't be
found?
On Mon, Jul 20, 2009 at 2:06 PM, Daniel Rice (דניאל
רייס)r...@google.com
/ProxyFactory.html.
Serializing those data in a String based representation could be potentially
harmful in term of bandwith and payload. Why not providing a way to store
this client data info in the HttpSession ?
Sami
On Mon, Jul 20, 2009 at 8:06 PM, Daniel Rice (דניאל רייס) r...@google.com
to ensure that you are stateless architecture
compliant. You will probably get complains coming from people that have a
pretty complex object graph to serialize.
Sami
On Mon, Jul 27, 2009 at 10:08 PM, Daniel Rice (דניאל רייס)
r...@google.com wrote:
Hi Sami -
1) To track the previous state would
Hi Ray -
You can take a look at my work-in-progress at
https://mondrian.corp.google.com/changelist/10864395. It's a retained-mode
API that translates into SVG or VML depending on browser
type. Retained-mode drawing uses a Graphics interface:
public interface Graphics {
ShapeElement
That could still theoretically fail if removeEldestEntry did something
weird that mutated the entries. But that seems pretty unlikely to be the
case.
Dan
On Mon, Aug 3, 2009 at 2:17 PM, John Tamplin j...@google.com wrote:
On Mon, Aug 3, 2009 at 2:13 PM, Daniel Rice (דניאל רייס)
r
with the interaction
between custom field serialization and subclassing?
Dan
On Mon, Aug 3, 2009 at 2:43 PM, BobV b...@google.com wrote:
On Mon, Aug 3, 2009 at 2:24 PM, Daniel Rice (דניאל רייס)r...@google.com
wrote:
That could still theoretically fail if removeEldestEntry did something
Hi John -
You say more generic but your comment seems to be suggesting greater
specificity (use versus how to use). Do you have some language in
mind?
Dan
On Tue, Sep 22, 2009 at 3:18 PM, j...@google.com wrote:
LGTM assuming you have tested it on all platforms, with minor nits.
O.K., I'll stick with the one-line version for non-Mac platforms.
On Tue, Sep 22, 2009 at 3:39 PM, John Tamplin j...@google.com wrote:
On Tue, Sep 22, 2009 at 3:27 PM, Daniel Rice (דניאל רייס) r...@google.com
wrote:
You say more generic but your comment seems to be suggesting greater
Is there something smaller than a full JDK-compatible String.format that
would work?
Dan
On Tue, Oct 20, 2009 at 11:32 AM, John Tamplin j...@google.com wrote:
On Tue, Oct 20, 2009 at 11:21 AM, Ray Ryan rj...@google.com wrote:
I'll bet a lot that the cross section of 40+ languages and
I'm not sure how to so this -- my testing involved manually setting the
machine's time zone. Ultimately the bug is in the fact that the native
Javascript Date functions deal with the missing hour differently that Java's
Date class, and I don't know that I can coerce the Javascript functions
They show up as severity=info which doesn't break the build. But at
least the @param names seem worth fixing...
Dan
On Tue, Nov 3, 2009 at 3:21 PM, rj...@google.com wrote:
Thanks.
Why is ant checkstyle passing for me?
http://gwt-code-reviews.appspot.com/89816
What is an SSW?
Dan
On Wed, Nov 4, 2009 at 2:24 PM, b...@google.com wrote:
LGTM, just nits.
http://gwt-code-reviews.appspot.com/90802/diff/1004/2001
File user/super/com/google/gwt/emul/java/util/Date.java (right):
http://gwt-code-reviews.appspot.com/90802/diff/1004/2001#newcode110
LGTM
On Wed, Nov 4, 2009 at 12:22 PM, j...@google.com wrote:
Reviewers: Ray Ryan, bruce, Dan Rice,
Description:
SwingWorker uses a pool of threads to run background tasks, and we
weren't actually using the background portion of this. Having multiple
threads processing SwingWorker calls
// future reserved words
abstract, int, short, boolean, interface, static, byte,
long, char, final, native, synchronized, float, package,
throws, goto, private, transient, implements, protected,
volatile, double, public,
What a future it will be...
Dan
On Tue, Nov 17, 2009 at 3:11 PM,
My thinking is that unless we provide an implementation of Double.toString,
we are best off trying to be consistent with the implementation at hand. If
we use Double.toString (i.e., just Javascript native double-to-string
conversion) some of the time, and use arithmetic ops to extract fractional
A quick and dirty test on Mac/Safari shows that the output of Hello
World! + 3.1415926535E-20 + + 123456789 is not sensitive to the system
locale setting. The numerical output uses a '.' and a lower-case 'e'.
Dan
On Tue, Nov 17, 2009 at 8:09 PM, Daniel Rice (דניאל רייס)
r
Window.setIconImages was introduced in Java 1.6. This doesn't compile on 1.5.
Dan
On Tue, Nov 17, 2009 at 5:52 PM, codesite-nore...@google.com wrote:
Revision: 6973
Author: j...@google.com
Date: Tue Nov 17 14:51:47 2009
Log: Adds additional icon sizes so Alt-Tab (etc) doesn't try and
No, what do I need to do for that?
Dan
On Thu, Dec 3, 2009 at 11:09 AM, robertvaw...@google.com wrote:
Did you test this case with the deRPC code base?
http://gwt-code-reviews.appspot.com/118802
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
No, I thought it would avoid a warning in Eclipse but Eclipse seems
fine with it.
Dan
On Tue, Dec 22, 2009 at 11:55 AM, j...@google.com wrote:
LGTM
http://gwt-code-reviews.appspot.com/126816/diff/1/7
File user/src/com/google/gwt/core/client/GWT.java (right):
Agreed, the worst case is that the error message will be less than fully
helpful.
On Thu, Jan 14, 2010 at 12:01 PM, j...@google.com wrote:
On 2010/01/14 16:02:34, Dan Rice wrote:
LGTM
I'm trying to think if this will encounter a problem in the Turkish
locale but I
think it should be
LGTM
On Tue, Jan 26, 2010 at 12:26 PM, jlaba...@google.com wrote:
Reviewers: Dan Rice,
Description:
As the title says.
http://code.google.com/p/google-web-toolkit/issues/detail?id=3757
Fix:
I swapped the row and column.
Testing:
I added a unit test to test this.
Please review this
What is 'com.google.gwt.*' in the file/dir list referring to?
Dan
On Tue, Mar 16, 2010 at 3:00 PM, rj...@google.com wrote:
Reviewers: Dan Rice,
Description:
Update bikeshed/eclipse.README based on feedback from Dan
Review by: r...@google.com
Please review this at
I did, checkstyle runs out of memory on the old version!
Dan
On Tue, Mar 16, 2010 at 4:13 PM, rj...@google.com wrote:
LGTM, presuming that you meant to include the symbols change in this
patch
http://gwt-code-reviews.appspot.com/218801
--
LGTM
On Tue, Mar 16, 2010 at 9:54 PM, rj...@google.com wrote:
Can you look again? I've made a few updates.
http://gwt-code-reviews.appspot.com/224801
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
I disagree with point (1). The APIs are not the same, just almost the
same. IMHO the builder should have a freeze method while the MutableArray
should not. This makes it clear that freezing is a build-time process. It
seems to me that this could be done with a little interface inheritance
implementation could still manage to share state between created instances
in some cases.
Dan
On Thu, Mar 25, 2010 at 1:58 PM, Bruce Johnson br...@google.com wrote:
On 3/25/10, Daniel Rice (דניאל רייס) r...@google.com wrote:
I disagree with point (1). The APIs are not the same, just almost
FYI this still needs review for the changes in IFrameLinker
(checkstyle) and PrecompressLinkerTest (removing use of
String.getBytes(Charset)).
Dan
On Mon, Mar 29, 2010 at 12:29 PM, r...@google.com wrote:
Reviewers: Lex,
Description:
Remove JDK 1.6-isms; fix a checkstyle warning
Please
LGTM
On Tue, Apr 20, 2010 at 4:39 AM, j...@google.com wrote:
Reviewers: Dan Rice,
Description:
Two small fixes:
- on my work laptop I would get lines way too big for the font size,
and on my home machine the underscores would get dropped because the
lines weren't quite large enough. I'm
Right now, the standard RPC format hasn't changed (although deRPC has,
since the whole point if to be able to eval the payload into the client).
So anyone who has a custom RPC system needs a way to generate a double[2]
from a long.
Longer term, I'd like to transmit longs as an int[3]
We thought about this, but the conclusion was that it would be better not
to expose yet another internal format. We do use this format for the deRPC
implementation.
Dan
On Thu, Jun 17, 2010 at 12:08 PM, cromwell...@gmail.com wrote:
If they're being eval'ed, why not transmit the long as a
, Ray Ryan rj...@google.com wrote:
Why not change deRPC?
On Thu, Jun 17, 2010 at 9:11 AM, Daniel Rice (דניאל רייס) r...@google.com
wrote:
We thought about this, but the conclusion was that it would be better
not to expose yet another internal format. We do use this format for the
deRPC
Is there any place that can be shared between dev and gwt-servlet.jar?
Dan
On Thu, Jun 17, 2010 at 4:18 PM, j...@google.com wrote:
LGTM, though the code duplication is ugly.
http://gwt-code-reviews.appspot.com/639801/show
--
Currently it's made by stripping things out of gwt-user.jar. I'll look
into how to add a class from dev to it.
Dan
On Thu, Jun 17, 2010 at 4:50 PM, John Tamplin j...@google.com wrote:
On Thu, Jun 17, 2010 at 4:19 PM, Daniel Rice (דניאל רייס) r...@google.com
wrote:
Is there any place
Looks like the code is still the same as what he is seeing -- I
haven't seen the stack trace and don't know how to repo it but the fix
seems reasonable.
Dan
On Mon, Jun 21, 2010 at 3:22 PM, Joel Webber j...@google.com wrote:
Thanks, Stephen.
@Dan: Is this still applicable, or has it been
On Fri, Jun 25, 2010 at 6:18 PM, t.bro...@gmail.com wrote:
Just like in plain JavaScript, the following would actually create an
ab property with no trace of char (tested in Safari 5 on Windows):
var o = {};
o['a\u2028b'] = 'c';
That's a bug in JavaScriptCore that can hardly be worked
We probably don't need these tests to run in full for every test cycle.
They would be useful, though, to run as smoke tests when new browser
versions are released. If there is a way to keep the test code around but
only invoke it infrequently I would be in favor of that.
Dan
On Tue, Jun 29,
Actually CellTreeNodeView is still work in progress and won't be submitted
as part of the current CL. Sorry for the confusion.
On Mon, Aug 2, 2010 at 12:14 PM, jlaba...@google.com wrote:
LGTM
http://gwt-code-reviews.appspot.com/710802/diff/1013/10009
File
Yes, I'll upload it in a sec.
Dan
On Thu, Aug 5, 2010 at 1:12 PM, Ray Ryan rj...@google.com wrote:
Ready for re-review?
On Wed, Aug 4, 2010 at 1:39 PM, r...@google.com wrote:
http://gwt-code-reviews.appspot.com/722802/show
--
I've had it off but I noticed in one of the changes this morning that an
@Override tag was (correctly) added and was curious enough to check the
setting. I would like to be able to enable warnings in my own Eclipse since
there are benefits to getting it right (namely avoiding accidental
Fortunately, we never actually extend Cell in any of the bikeshed code.
Dan
On Mon, Aug 30, 2010 at 2:55 PM, jlaba...@google.com wrote:
I'm pretty sure we extend Cell in the Expenses sample, and in the
Scaffold and Cookbook (what remains) in bikeshed. Please make sure to
update these too.
This isn't really working in prod mode so you can hold off on reviewing if
you'd prefer.
Dan
On Fri, Sep 3, 2010 at 4:16 PM, r...@google.com wrote:
Reviewers: jlabanca,
Description:
Implement keyboard navigation for CellBrowser
Please review this at
Sorry, I took the lead from the pre-existing comment on line 35...
Dan
On Fri, Sep 17, 2010 at 12:36 PM, rj...@google.com wrote:
no suffix, please. It should be PlaceChangeRequest
On 2010/09/17 16:32:11, rice wrote:
http://gwt-code-reviews.appspot.com/894801/show
--
Fixed and re-uploaded.
On Fri, Sep 17, 2010 at 12:36 PM, rj...@google.com wrote:
no suffix, please. It should be PlaceChangeRequest
On 2010/09/17 16:32:11, rice wrote:
http://gwt-code-reviews.appspot.com/894801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Somehow a class like P extends BlahBlahBlah was being treated as
real class, resulting in a generated PImpl class which broke all
kinds of ways. I need to put a breakpoint at that line to see if it
still happens and try to deal with the issue further upstream.
Dan
On Wed, Sep 29, 2010 at 7:58
This emulates JRE behavior, as pointed out in
http://code.google.com/p/google-web-toolkit/issues/detail?id=5297.
Dan
On Tue, Oct 5, 2010 at 5:11 PM, j...@google.com wrote:
http://gwt-code-reviews.appspot.com/962801/diff/1/2
File
I do plan to rewrite it, but this should unbreak the windows build
for the time being.
On Mon, Oct 11, 2010 at 4:14 PM, j...@google.com wrote:
Why not just rewrite the shell script as a Java app, like we did for
WebAppCreator/etc?
If not, I would prefer to just have the static file used
LGTM
Can you add a comment to that effect
On Thu, Oct 14, 2010 at 10:09 AM, jlaba...@google.com wrote:
http://gwt-code-reviews.appspot.com/997801/diff/1/3
File user/src/com/google/gwt/user/cellview/client/CellTree.java (right):
http://gwt-code-reviews.appspot.com/997801/diff/1/3#newcode479
Made changes and re-uploaded to rietveld
On Thu, Oct 14, 2010 at 12:58 PM, b...@google.com wrote:
Thanks for doing this.
LGTM, with nits.
http://gwt-code-reviews.appspot.com/1002801/diff/1/2
File user/src/com/google/gwt/editor/client/AutoBean.java (right):
Sure, consistency Я we
On Mon, Oct 18, 2010 at 6:13 PM, Ray Ryan rj...@google.com wrote:
If you're really up for it Dan, us ♥ consistency.
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
That would work for this package, but other HTML5 stuff involves
additions to core GWT classes, unlike the situation we had with cell
widgets last time. It seems like it would be tough for stuff in
gwt-user to have dependencies on bikeshed, and even tougher to avoid
such dependencies while
I saw 4.5% on the uncompressed size.
Dan
On Tue, Jan 18, 2011 at 4:39 PM, Ray Cromwell cromwell...@google.com wrote:
I'm curious, has there been any measurement/data as to the effect this
has on output size?
On Mon, Jan 17, 2011 at 1:18 PM, sco...@google.com wrote:
LGTM, just nits. No
Here's a manual diff. Is there some slick way to upload it to
Mondrian or Rietveld that won't make them confused?
Dan
On Thu, Jan 20, 2011 at 3:52 PM, sco...@google.com wrote:
Can you upload a diff relative to what you checked in before you rolled
it back? I just want to see what you
I spent a few days attempting this -- it's not so simple. For example:
1) Enums can't extend Enums. Several public types contain nested Enums.
2) If a type oldA extends newA, and a method in it returns an instance of
oldA, you cant simply delegate to the implementation in newA since that will
I did the reimplementation. Being less familiar with the intricacies of
JavaScript, it seems likely that I missed an opportunity to preserve the
nativeness of the array. I'd be happy to take a look at this after the 2.3
release.
On Mon, Mar 28, 2011 at 5:52 PM, Scott Blum sco...@google.com
Yep.
Dan
On Sun, May 1, 2011 at 8:00 PM, jlaba...@google.com wrote:
@rice -
Will you have time to review this on Monday? I'd like to get it
submitted this week.
On 2011/04/27 18:14:47, Jeff Larsen wrote:
On 2011/04/26 21:10:21, jlabanca wrote:
On Tue, Apr 26, 2011 at 4:57 PM, Jeff
66 matches
Mail list logo