Done, submitting.

http://gwt-code-reviews.appspot.com/710801/diff/18001/19001
File
bikeshed/src/com/google/gwt/sample/expenses/gwt/request/ExpensesRequestFactory.java
(left):

http://gwt-code-reviews.appspot.com/710801/diff/18001/19001#oldcode50
bikeshed/src/com/google/gwt/sample/expenses/gwt/request/ExpensesRequestFactory.java:50:
// }
Nope, still not an unrelated change. Nuked on purpose.

http://gwt-code-reviews.appspot.com/710801/diff/18001/19023
File
samples/dynatablerf/src/com/google/gwt/sample/dynatablerf/shared/PersonRecord.java
(right):

http://gwt-code-reviews.appspot.com/710801/diff/18001/19023#newcode27
samples/dynatablerf/src/com/google/gwt/sample/dynatablerf/shared/PersonRecord.java:27:
public interface PersonRecord extends Record {
On 2010/07/23 19:58:57, bobv wrote:
Rename to PersonProxy.

Done.

http://gwt-code-reviews.appspot.com/710801/diff/18001/19028
File
user/src/com/google/gwt/requestfactory/rebind/RequestFactoryGenerator.java
(right):

http://gwt-code-reviews.appspot.com/710801/diff/18001/19028#newcode244
user/src/com/google/gwt/requestfactory/rebind/RequestFactoryGenerator.java:244:
for (JMethod method : interfaceType.getMethods()) {
On 2010/07/23 19:58:57, bobv wrote:
Do you want to use getOverridableMethods() to inherit methods from
super-interfaces?

Done.

http://gwt-code-reviews.appspot.com/710801/diff/18001/19028#newcode380
user/src/com/google/gwt/requestfactory/rebind/RequestFactoryGenerator.java:380:
for (JMethod method : selectorInterface.getMethods()) {
On 2010/07/23 19:58:57, bobv wrote:
Methods from super-interfaces?

Done.

http://gwt-code-reviews.appspot.com/710801/show

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

Reply via email to