[gwt-contrib] apicheck should run on Windows

2009-07-10 Thread fabbott

Reviewers: amitmanjhi,

Description:
the old form got confused by a refJar "path" of
'C:\GWT\tools\api-checker\reference\gwt-dev-modified.jar:C:\GWT\tools\api-checker\reference\gwt-user-modified.jar'
because splitting on colons was wrong.  Worse, ant thought that it had
succeeded because an arg-parsing error was caught, help printed, and
ultimately 0 exited.

The new form splits on the path.separator system propery instead, uses
ant's  to input either a : or ; separated value depending
on OS, and catches arg parse failures as non-zero return.

Please review this at http://gwt-code-reviews.appspot.com/51801

Affected files:
   build.xml

tools/api-checker/src/com/google/gwt/tools/apichecker/ApiCompatibilityChecker.java



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



[gwt-contrib] [google-web-toolkit commit] r5717 - More ant tweaks, this time to speed dev and jni tests by not attempting to test the OS'es...

2009-07-10 Thread codesite-noreply

Author: fabb...@google.com
Date: Fri Jul 10 19:19:32 2009
New Revision: 5717

Modified:
trunk/dev/build.xml
trunk/jni/build.xml

Log:
More ant tweaks, this time to speed dev and jni tests by not attempting to  
test the OS'es we're not on.

Review by: rjrjr

Modified: trunk/dev/build.xml
==
--- trunk/dev/build.xml (original)
+++ trunk/dev/build.xml Fri Jul 10 19:19:32 2009
@@ -3,6 +3,13 @@



+
+  
+
+
+
+  
+

  
  

Modified: trunk/jni/build.xml
==
--- trunk/jni/build.xml (original)
+++ trunk/jni/build.xml Fri Jul 10 19:19:32 2009
@@ -2,4 +2,9 @@



+
+  
+
+
+  
  

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



[gwt-contrib] [google-web-toolkit commit] r5716 - Ant changes to make top-level "test" also exercise checkstyle and apicheck. This thus su...

2009-07-10 Thread codesite-noreply

Author: fabb...@google.com
Date: Fri Jul 10 19:18:10 2009
New Revision: 5716

Modified:
trunk/build.xml
trunk/common.ant.xml

Log:
Ant changes to make top-level "test" also exercise checkstyle and  
apicheck.  This thus subsumes "prebuild", which people don't know to use  
and is thus doomed to go away.

Review by: rjrjr

Modified: trunk/build.xml
==
--- trunk/build.xml (original)
+++ trunk/build.xml Fri Jul 10 19:18:10 2009
@@ -118,7 +118,8 @@
  


-  
+  
  
  
  
@@ -130,7 +131,8 @@
  


-  
+





-  

  

Modified: trunk/common.ant.xml
==
--- trunk/common.ant.xml(original)
+++ trunk/common.ant.xmlFri Jul 10 19:18:10 2009
@@ -276,6 +276,7 @@
  
  

+  


  

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



[gwt-contrib] Re: "test" in dev & jni are overly aggressive

2009-07-10 Thread rjrjr

LGTM

http://gwt-code-reviews.appspot.com/47812

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



[gwt-contrib] Re: allow checkstyle to work in unbuilt samples

2009-07-10 Thread Ray Ryan
LGTM
And I'll give you a yell when it's succeeded in my client.

On Fri, Jul 10, 2009 at 4:23 PM, Freeland Abbott  wrote:

> Revised for the slightly larger goal of "allow 'ant clean presubmit'" to
> work, not merely checkstyle... apicheck requires that tools be built, and
> the more minimalist targets buildonly and dist-dev do not do that.
>
>
>
> On Fri, Jul 10, 2009 at 3:25 PM, Freeland Abbott wrote:
>
>> As people use dist-dev, they're more likely to have samples unbuilt (i.e.
>> no build/out/samples/...).As checkstyle is now fatal, and becoming part
>> of top-level tests, checkstyle will tend to run in unbuilt samples.
>> Today, this is Bad.  But with this fix, it may become Happy.  Or it may
>> Explode.
>>
>> All eyes are on you, Ray... no pressure.
>>
>>
>>
>> Index: common.ant.xml
>> ===
>> --- common.ant.xml  (revision 5714)
>> +++ common.ant.xml  (working copy)
>> @@ -276,6 +276,7 @@
>>  
>>  
>>> classpath="${gwt.tools.antlib}/checkstyle-all-4.2.jar;${gwt.build.lib}/gwt-customchecks.jar"
>> />
>> +  
>>> config="${gwt.root}/eclipse/settings/code-style/gwt-checkstyle.xml"
>> maxErrors="0" failOnViolation="false"
>> failureProperty="gwt.checkstyle.failed">
>> > toFile="@{outputdirectory}/checkstyle_log.xml"/>
>>  > file="${gwt.root}/eclipse/settings/code-style/google.header" />
>>
>>
>

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



[gwt-contrib] Re: allow checkstyle to work in unbuilt samples

2009-07-10 Thread Freeland Abbott
Revised for the slightly larger goal of "allow 'ant clean presubmit'" to
work, not merely checkstyle... apicheck requires that tools be built, and
the more minimalist targets buildonly and dist-dev do not do that.


On Fri, Jul 10, 2009 at 3:25 PM, Freeland Abbott  wrote:

> As people use dist-dev, they're more likely to have samples unbuilt (i.e.
> no build/out/samples/...).As checkstyle is now fatal, and becoming part of
> top-level tests, checkstyle will tend to run in unbuilt samples.
> Today, this is Bad.  But with this fix, it may become Happy.  Or it may
> Explode.
>
> All eyes are on you, Ray... no pressure.
>
>
>
> Index: common.ant.xml
> ===
> --- common.ant.xml  (revision 5714)
> +++ common.ant.xml  (working copy)
> @@ -276,6 +276,7 @@
>  
>  
> classpath="${gwt.tools.antlib}/checkstyle-all-4.2.jar;${gwt.build.lib}/gwt-customchecks.jar"
> />
> +  
> config="${gwt.root}/eclipse/settings/code-style/gwt-checkstyle.xml"
> maxErrors="0" failOnViolation="false"
> failureProperty="gwt.checkstyle.failed">
>  toFile="@{outputdirectory}/checkstyle_log.xml"/>
>   file="${gwt.root}/eclipse/settings/code-style/google.header" />
>
>

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



clean-presub.patch
Description: Binary data


[gwt-contrib] "test" in dev & jni are overly aggressive

2009-07-10 Thread fabbott

Reviewers: Ray Ryan,

Description:
As long as it's ant day... this is a performance tweak.  Right now, when
we test in the native-code directories (dev and jni), we nominally test
all three operating systems, which can result in triplicate builds of
alldeps.jar.  Since we, in fact, can only test for *this* OS that we're
on, there's no reason to pretend we can "-do" the full fanout.

Please review this at http://gwt-code-reviews.appspot.com/47812

Affected files:
   dev/build.xml
   jni/build.xml


Index: dev/build.xml
===
--- dev/build.xml   (revision 5695)
+++ dev/build.xml   (working copy)
@@ -3,6 +3,13 @@



+
+  
+
+
+
+  
+

  
  
Index: jni/build.xml
===
--- jni/build.xml   (revision 5695)
+++ jni/build.xml   (working copy)
@@ -2,4 +2,9 @@



+
+  
+
+
+  
  



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



[gwt-contrib] allow checkstyle to work in unbuilt samples

2009-07-10 Thread Freeland Abbott
As people use dist-dev, they're more likely to have samples unbuilt (i.e. no
build/out/samples/...).As checkstyle is now fatal, and becoming part of
top-level tests, checkstyle will tend to run in unbuilt samples.
Today, this is Bad.  But with this fix, it may become Happy.  Or it may
Explode.

All eyes are on you, Ray... no pressure.



Index: common.ant.xml
===
--- common.ant.xml  (revision 5714)
+++ common.ant.xml  (working copy)
@@ -276,6 +276,7 @@
 
 
   
+  
   

 

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



[gwt-contrib] Re: Incubator API - Date/Time Handling

2009-07-10 Thread Carver

BTW, when you start working on the TimePicker, I would be happy to
lend a hand.  I would start right away, but I have never sent patches
to GWT.  So I would prefer to pair-code with someone who has, first.

On Jul 10, 3:03 pm, Carver  wrote:
> The TimePicker Demo in the following project has a pretty elegant two-
> click solution: (it claims to be modeling jquery)
>
> http://code.google.com/p/gwt-incubator-lib/
>
> Note that this seems unaffiliated with the "main" gwt incubator.  I am
> not involved with it, and the code looks dormant (no edits since
> February).  If nothing else, I would love to see the UI design
> borrowed.  I find the current incubator time picker to be fairly
> frustrating because of how much "mouse -> keyboard -> mouse" is
> required.
>
> On Jul 9, 6:18 pm, dflorey  wrote:
>
>
>
> > I don't remember if I already ported the spinner widgets to the new
> > handler system...but I guess so.
> > The TimePicker is a very simple widget that simply combines different
> > spinners to enter or display a given time.
> > While it is working well I'm not quite sure if it is the best way to
> > enter a time in general.
> > It would be worth to have a look at different webapps to find the best
> > approach how to enter dates / times and adjust the affected widgets
> > (DatePicker,TimePicker,DateTimePicker).
> > In Google contacts the birthday can be selected by simple dropdown
> > boxes (and this is very easy compared to enter your birthday with gwt
> > datepicker...).
> > In a few weeks I'll have some time to take a look at these widgets if
> > appreciated.
>
> > On Jul 9, 5:16 pm, Joel Webber  wrote:
>
> > > Anyone know what the status of the incubator TimePicker is? I haven't 
> > > looked
> > > in a while.
>
> > > On Thu, Jul 9, 2009 at 10:59 AM, Paul Siml  wrote:
> > > >  Hi,
>
> > > > I am looking for some information on the TimePicker in the incubator.
> > > > Reading tacking issues I found a time picker but there does not appear 
> > > > to be
> > > > a corresponding Incubator API entry.  In addition, the depreciated
> > > > widgetideas DateTimePicker API links are broken in the API as well.
>
> > > > com.google.gwt.gen2.picker.client.TimePicker;
> > > > Please let me know who I should contact.
>
> > > > Thanks Paul
> > > > Siml Systems, LLC
--~--~-~--~~~---~--~~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~--~~~~--~~--~--~---



[gwt-contrib] Re: Incubator API - Date/Time Handling

2009-07-10 Thread Carver

The TimePicker Demo in the following project has a pretty elegant two-
click solution: (it claims to be modeling jquery)

http://code.google.com/p/gwt-incubator-lib/

Note that this seems unaffiliated with the "main" gwt incubator.  I am
not involved with it, and the code looks dormant (no edits since
February).  If nothing else, I would love to see the UI design
borrowed.  I find the current incubator time picker to be fairly
frustrating because of how much "mouse -> keyboard -> mouse" is
required.

On Jul 9, 6:18 pm, dflorey  wrote:
> I don't remember if I already ported the spinner widgets to the new
> handler system...but I guess so.
> The TimePicker is a very simple widget that simply combines different
> spinners to enter or display a given time.
> While it is working well I'm not quite sure if it is the best way to
> enter a time in general.
> It would be worth to have a look at different webapps to find the best
> approach how to enter dates / times and adjust the affected widgets
> (DatePicker,TimePicker,DateTimePicker).
> In Google contacts the birthday can be selected by simple dropdown
> boxes (and this is very easy compared to enter your birthday with gwt
> datepicker...).
> In a few weeks I'll have some time to take a look at these widgets if
> appreciated.
>
> On Jul 9, 5:16 pm, Joel Webber  wrote:
>
>
>
> > Anyone know what the status of the incubator TimePicker is? I haven't looked
> > in a while.
>
> > On Thu, Jul 9, 2009 at 10:59 AM, Paul Siml  wrote:
> > >  Hi,
>
> > > I am looking for some information on the TimePicker in the incubator.
> > > Reading tacking issues I found a time picker but there does not appear to 
> > > be
> > > a corresponding Incubator API entry.  In addition, the depreciated
> > > widgetideas DateTimePicker API links are broken in the API as well.
>
> > > com.google.gwt.gen2.picker.client.TimePicker;
> > > Please let me know who I should contact.
>
> > > Thanks Paul
> > > Siml Systems, LLC
--~--~-~--~~~---~--~~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~--~~~~--~~--~--~---



[gwt-contrib] [google-web-toolkit commit] r5714 - Added warning that latest CheckStyle doesn't work

2009-07-10 Thread codesite-noreply

Author: rj...@google.com
Date: Fri Jul 10 09:14:42 2009
New Revision: 5714

Modified:
trunk/eclipse/README.txt

Log:
Added warning that latest CheckStyle doesn't work
for us.



Modified: trunk/eclipse/README.txt
==
--- trunk/eclipse/README.txt(original)
+++ trunk/eclipse/README.txtFri Jul 10 09:14:42 2009
@@ -1,4 +1,4 @@
-Eclipse 3.3.X instructions
+Eclipse 3.X instructions

  These instructions are intended for contributors to the GWT source
  code repository that want to run the Eclipse IDE. It describes how to
@@ -105,7 +105,7 @@

  Checkstyle is used to enforce good programming style.

-1. Install Checkstyle
+1. Install Checkstyle version 4.4.2 (newer versions will not work)

  The Eclipse Checkstyle plugin can be found at:
http://eclipse-cs.sourceforge.net/

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



[gwt-contrib] [google-web-toolkit commit] r5713 - Revert c5711 due to check style failures.

2009-07-10 Thread codesite-noreply

Author: rj...@google.com
Date: Fri Jul 10 08:13:58 2009
New Revision: 5713

Removed:
trunk/user/src/com/google/gwt/junit/FakeMessagesMaker.java
trunk/user/test/com/google/gwt/junit/FakeMessagesMakerTest.java
trunk/user/test/com/google/gwt/junit/JUnitSuite.java
Modified:
trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java
trunk/user/test/com/google/gwt/user/client/ui/HTMLPanelTest.java

Log:
Revert c5711 due to check style failures.



Modified: trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java
==
--- trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java (original)
+++ trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java Fri Jul 10  
08:13:58 2009
@@ -40,29 +40,16 @@
}

/**
-   * Creates an HTML panel with the specified HTML contents inside a DIV
-   * element. Any element within this HTML that has a specified id can  
contain a
-   * child widget.
+   * Creates an HTML panel with the specified HTML contents. Any element  
within
+   * this HTML that has a specified id can contain a child widget.
 *
 * @param html the panel's HTML
 */
public HTMLPanel(String html) {
-this("div", html);
-  }
-
-  /**
-   * Creates an HTML panel whose root element has the given tag, and with  
the
-   * specified HTML contents. Any element within this HTML that has a  
specified
-   * id can contain a child widget.
-   *
-   * @param tag the tag of the root element
-   * @param html the panel's HTML
-   */
-  public HTMLPanel(String tag, String html) {
-setElement(DOM.createElement(tag));
+setElement(DOM.createDiv());
  DOM.setInnerHTML(getElement(), html);
}
-
+
/**
 * Adds a child widget to the panel, contained within the HTML element
 * specified by a given id.

Modified: trunk/user/test/com/google/gwt/user/client/ui/HTMLPanelTest.java
==
--- trunk/user/test/com/google/gwt/user/client/ui/HTMLPanelTest.java 
(original)
+++ trunk/user/test/com/google/gwt/user/client/ui/HTMLPanelTest.javaFri  
Jul 10 08:13:58 2009
@@ -1,12 +1,12 @@
  /*
   * Copyright 2008 Google Inc.
- *
+ *
   * Licensed under the Apache License, Version 2.0 (the "License"); you may  
not
   * use this file except in compliance with the License. You may obtain a  
copy of
   * the License at
- *
+ *
   * http://www.apache.org/licenses/LICENSE-2.0
- *
+ *
   * Unless required by applicable law or agreed to in writing, software
   * distributed under the License is distributed on an "AS IS" BASIS,  
WITHOUT
   * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
@@ -117,28 +117,6 @@

  Node next = button.getElement().getNextSibling();
  assertEquals("bar", next.getNodeValue());
-  }
-
-  /**
-   * Tests arbitrary root tag
-   */
-  public void testCustomRootTag() {
-HTMLPanel hp = new HTMLPanel("table", "Hello ");
-InlineLabel label = new InlineLabel("World");
-hp.addAndReplaceElement(label, "labelHere");
-
-Element parent = label.getElement().getParentElement();
-assertEquals("td", parent.getTagName().toLowerCase());
-
-parent = parent.getParentElement();
-assertEquals("tr", parent.getTagName().toLowerCase());
-
-while (parent != null && parent != hp.getElement()) {
-  parent = parent.getParentElement();
-}
-
-assertNotNull(parent);
-assertEquals("table", parent.getTagName().toLowerCase());
}

/**

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



[gwt-contrib] Comment on OophmForMacBasedGwtContributors in google-web-toolkit

2009-07-10 Thread codesite-noreply

Comment by tamplinjohn:

Typo -- the correct target is dist-dev, not dev-dist.

See eclipse/README.txt for setting up Eclipse for working with GWT.

The Google Eclipse plugin doesn't have any support for using an Eclipse  
project for GWT currently (you have to point it at a GWT distribution  
directory), so I believe you will have to actually unjar gwt-dev-mac.jar,  
rm -rf org/eclipse/swt, and rejar it.  I don't use a Mac so I haven't  
tested it.


For more information:
http://code.google.com/p/google-web-toolkit/wiki/OophmForMacBasedGwtContributors

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



[gwt-contrib] [google-web-toolkit commit] r5712 - Edited wiki page through web user interface.

2009-07-10 Thread codesite-noreply

Author: tamplinjohn
Date: Fri Jul 10 07:11:25 2009
New Revision: 5712

Modified:
wiki/OophmForMacBasedGwtContributors.wiki

Log:
Edited wiki page through web user interface.

Modified: wiki/OophmForMacBasedGwtContributors.wiki
==
--- wiki/OophmForMacBasedGwtContributors.wiki   (original)
+++ wiki/OophmForMacBasedGwtContributors.wiki   Fri Jul 10 07:11:25 2009
@@ -11,7 +11,7 @@
 * Make sure your oophm jar is up to date
* cd trunk
* ant clean
-  * ant dev-dist
+  * ant dist-dev
 * Back in eclipse, go to the Package Explorer, right click gwt-dev-mac  
and choose Properties
 * Java Build Path / Libraries
 * Delete the GWT_TOOLS/lib/eclipse/org.eclipse.swt.carbon-macosx... jar

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



[gwt-contrib] Re: Custom tag constructor for HTMLPanel

2009-07-10 Thread rjrjr

Committed revision 5711.

On 2009/07/09 00:02:01, Ray Ryan wrote:
> Forgot about this during my vacation. In it goes.

> Committed revision 5694.

> On 2009/06/12 21:22:50, jgw wrote:
> > On 2009/06/12 21:20:50, jgw wrote:
> > > Ray:
> > > That won't really protect anyone from anything, as they could just
call
> > > DOM.createElement("bugger") and hand that in. Also, accepting an
existing
> > > element is another way to say HTMLPanel.wrap(),  which we haven't
provided
> so
> > > far--you really want to go there?
> > >
> > > Anyhow, does any browser not accept 
> > > 
> >
> > Well, we *do* have lots of methods to try and keep people 'on the
rails', such
> > as createDivElement(), etc. which I believe reduces the risk of
errors in
> > practice (they can always call createElement("bugger"), but usually
don't).
> But
> > I think I'm ok with it, as it at least reduces the probability that
someone
> will
> > try and use this as a back-door wrap() method, which would lead to
much
> wailing
> > and gnashing of teeth on my part.
> >
> > LGTM.



http://gwt-code-reviews.appspot.com/39802

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



[gwt-contrib] [google-web-toolkit commit] r5711 - HTMLPanel now allows an arbitrary element to be set as its root, new

2009-07-10 Thread codesite-noreply

Author: rj...@google.com
Date: Fri Jul 10 07:02:01 2009
New Revision: 5711

Added:
trunk/user/src/com/google/gwt/junit/FakeMessagesMaker.java
trunk/user/test/com/google/gwt/junit/FakeMessagesMakerTest.java
trunk/user/test/com/google/gwt/junit/JUnitSuite.java
Modified:
trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java
trunk/user/test/com/google/gwt/user/client/ui/HTMLPanelTest.java

Log:
HTMLPanel now allows an arbitrary element to be set as its root, new
constructor HTMLPanel(String tag, String html)

   http://gwt-code-reviews.appspot.com/39802

Introduces FakeMessagesMaker, to allow JUnit (non-GWTTestCase) testing
of objects that rely on generated Messages objects.

   http://gwt-code-reviews.appspot.com/48809



Added: trunk/user/src/com/google/gwt/junit/FakeMessagesMaker.java
==
--- (empty file)
+++ trunk/user/src/com/google/gwt/junit/FakeMessagesMaker.java  Fri Jul 10  
07:02:01 2009
@@ -0,0 +1,48 @@
+package com.google.gwt.junit;
+
+import com.google.gwt.i18n.client.Messages;
+
+import java.lang.reflect.InvocationHandler;
+import java.lang.reflect.Method;
+import java.lang.reflect.Proxy;
+import java.util.Arrays;
+
+/**
+ * Helper to make a fake implementation of any {...@link Messages} interface  
via
+ * reflection, for use in JUnit tests. (This will not work in  
GWTTestCase.) All
+ * calls to the returned object return the method name followed by the  
passed
+ * parameters as a list surrounded by [].
+ * 
+ * Note that the default message text is very consciously not made  
available
+ * through the fake, to help tests ensure that specific translations of
+ * localized text are not relied upon.
+ * 
+ * Sample use:
+ *
+ * interface MyMessages extends Messages {
+ *   @DefaultMessage("Isn''t this the fakiest?")
+ *   @Description("A sample message to be tested.")
+ *   String myMessage();
+ * }
+ *
+ * public void testSimple() {
+ *  MyMessages messages = FakeMessagesMaker.create(MyMessages.class);
+ *  assertEquals("myMessage", messages.myMessage());
+ * }
+ * 
+ */
+public class FakeMessagesMaker implements InvocationHandler {
+  public static  T create(Class messagesClass) {
+return messagesClass.cast(Proxy.newProxyInstance(
+FakeMessagesMaker.class.getClassLoader(), new Class[]  
{messagesClass},
+new FakeMessagesMaker()));
+  }
+
+  public Object invoke(Object proxy, Method method, Object[] args)
+  throws Throwable {
+String name = method.getName();
+
+return (args == null || args.length == 0) ? name : name
++ Arrays.asList(args);
+  }
+}

Modified: trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java
==
--- trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java (original)
+++ trunk/user/src/com/google/gwt/user/client/ui/HTMLPanel.java Fri Jul 10  
07:02:01 2009
@@ -40,16 +40,29 @@
}

/**
-   * Creates an HTML panel with the specified HTML contents. Any element  
within
-   * this HTML that has a specified id can contain a child widget.
+   * Creates an HTML panel with the specified HTML contents inside a DIV
+   * element. Any element within this HTML that has a specified id can  
contain a
+   * child widget.
 *
 * @param html the panel's HTML
 */
public HTMLPanel(String html) {
-setElement(DOM.createDiv());
-DOM.setInnerHTML(getElement(), html);
+this("div", html);
}

+  /**
+   * Creates an HTML panel whose root element has the given tag, and with  
the
+   * specified HTML contents. Any element within this HTML that has a  
specified
+   * id can contain a child widget.
+   *
+   * @param tag the tag of the root element
+   * @param html the panel's HTML
+   */
+  public HTMLPanel(String tag, String html) {
+setElement(DOM.createElement(tag));
+DOM.setInnerHTML(getElement(), html);
+  }
+
/**
 * Adds a child widget to the panel, contained within the HTML element
 * specified by a given id.

Added: trunk/user/test/com/google/gwt/junit/FakeMessagesMakerTest.java
==
--- (empty file)
+++ trunk/user/test/com/google/gwt/junit/FakeMessagesMakerTest.java Fri Jul 
 
10 07:02:01 2009
@@ -0,0 +1,29 @@
+package com.google.gwt.junit;
+
+import com.google.gwt.i18n.client.Messages;
+
+import junit.framework.TestCase;
+
+public class FakeMessagesMakerTest extends TestCase {
+  interface MyMessages extends Messages {
+@DefaultMessage("Isn''t this the fakiest?")
+@Description("A sample message to be tested.")
+String myMessage();
+
+@DefaultMessage("Isn''t this the fakiest? Pick one: {1} or {2}?")
+@Description("A sample message with parameters.")
+String myArgumentedMessage(@Example("yes") String yes,
+@Example("no") String no);
+  }
+
+  public void testSimple() {
+MyMessages messages = FakeMessag

[gwt-contrib] Re: suboptimal obfuscation for compression?

2009-07-10 Thread Joel Webber
Good find. It sounds like a pretty simple change to allocate these more
sensibly.

On Fri, Jul 10, 2009 at 4:49 AM, Ray Cromwell  wrote:

>
> Looking at the output of the compiled Mail application, I noticed the
> following property of function declarations:
>
> function dM(d,b){var a,c;if(b<0){throw AZ(new
> zZ(),ij+b)}c=d.a.rows.length;for(a=c;a<=b;++a){pN(d,a)}}
> function eM(f,d,c){var e=f.rows[d];for(var b=0;b a=$doc.createElement(gi);e.appendChild(a)}}
> function pM(a){while(++a.a function qZ(){return this.$H||(this.$H=++mp)}
> function
> oU(d,a,c,b){if(!b||b==c){return}oU(d,a,c,mq((Ep(),b)));ey(a.a,a.b++,b)}
> function rU(i,a,e,h)
>
> note that the function params are assigned single character symbols, but
> they are not consistently assigned and in the same order. If instead, the
> first arguments of any function were a, b, c, ... appear in order (they
> could be any reserved sequence), then characters sequences like
>
> (a){
> (a,b){
> (a,b,c){
> (a,b,c,d){
>
> would appear frequently and yield more backreferences in the LZ77 portion
> of the DEFLATE algorithm, plus, they would occur far more frequently, giving
> a smaller bit representation in the huffman tables. This might lead to
> significant savings when gzipping.
>
> -Ray
>
>
> >
>

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



[gwt-contrib] Comment on OophmForMacBasedGwtContributors in google-web-toolkit

2009-07-10 Thread codesite-noreply

Comment by brett.wooldridge:

I should note that it kind of works on my Mac. However, over the GWT Hosted  
Mode window is a constant pinwheel cursor. I can't click on anything. I  
would have thought maybe this is due to the Swing client, but there  
certainly is no -XstartOnFirstThread? in my arguments. Is it because of SWT  
(e.g. step 4 was not possible because I don't have GWT in my workspace)?

Is there a guide anywhere to setting up the GWT source in Eclipse itself.  
There don't appear to be top-level eclipse projects. Do I need to generate  
projects form ant files etc?


For more information:
http://code.google.com/p/google-web-toolkit/wiki/OophmForMacBasedGwtContributors

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



[gwt-contrib] Comment on OophmForMacBasedGwtContributors in google-web-toolkit

2009-07-10 Thread codesite-noreply

Comment by brett.wooldridge:

I should note that it kind of works on my Mac.  However, over the GWT  
Hosted Mode window is a constant pinwheel cursor.  I can't click on  
anything.  I would have thought maybe this is due to the Swing client, but  
there certainly is no -XstartOnFirstThread in my arguments.  Is it because  
of SWT (e.g. step 4 was not possible because I don't have GWT in my  
workspace)?

Is there a guide anywhere to setting up the GWT source in Eclipse itself.   
There don't appear to be top-level eclipse projects.  So I need to generate  
projects form ant files etc?


For more information:
http://code.google.com/p/google-web-toolkit/wiki/OophmForMacBasedGwtContributors

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



[gwt-contrib] Comment on OophmForMacBasedGwtContributors in google-web-toolkit

2009-07-10 Thread codesite-noreply

Comment by brett.wooldridge:

Thanks for the write-up.  Couple of things.

I'm checked out from the truck, but there is no "dev-dist" target.  A  
simple 'ant' seems to suffice for building gwt, though possibly building  
more than necessary.

I'm using the Eclipse plugin from the main site.  Should I be building this  
too?

Missing step above?  Go to Preferences->Google->Web Toolkit and add the  
toolkit from the trunk.  GWT itself is not checked out in my Eclipse  
workspace.  Should it be?  I checked it out and built on the command line  
only.  As a result, step 2,3,4 don't make much sense, making me think your  
recipe is indeed describing a workspace where GWT is checked out along-side  
my project.  Correct?




For more information:
http://code.google.com/p/google-web-toolkit/wiki/OophmForMacBasedGwtContributors

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



[gwt-contrib] suboptimal obfuscation for compression?

2009-07-10 Thread Ray Cromwell
Looking at the output of the compiled Mail application, I noticed the
following property of function declarations:

function dM(d,b){var a,c;if(b<0){throw AZ(new
zZ(),ij+b)}c=d.a.rows.length;for(a=c;a<=b;++a){pN(d,a)}}
function eM(f,d,c){var e=f.rows[d];for(var b=0;bhttp://groups.google.com/group/Google-Web-Toolkit-Contributors
-~--~~~~--~~--~--~---