On Wed, Dec 10, 2008 at 6:51 PM, BobV <[EMAIL PROTECTED]> wrote:

>  dev/core/src/com/google/gwt/core/ext/linker/AbstractLinker.java
>      |   71    70 +    1 -     0 !
>
> What's with the @SuppressWarnings tags?
>

logger is unused, UnableToComplete not actually thrown.


>
>  dev/core/src/com/google/gwt/core/ext/linker/impl/SelectionScriptLinker.java
> |   20  13 +    7 -     0 !
>
> emitSelectionScript(): add a comment to explain logic
> L233: break string literal
>

Done.


>
>  dev/core/src/com/google/gwt/core/ext/linker/impl/StandardLinkerContext.java
> |   18  11 +    7 -     0 !
>
> Constructor: That one-liner comment was intended to prevent the reader
> from wondering when generated artifacts are included in the SLC.
> produceOutputDirectory(): Should that be a <= instead of < ?
>

Reverted comment erasure; changed to <=


>  dev/core/src/com/google/gwt/core/linker/IFrameLinker.java
>      |   36    32 +    4 -     0 !
>
> Extract a URLArtifact type?
>

Added a TODO in the spirit of 0, 1, infinity.


>
>  dev/core/src/com/google/gwt/core/linker/IFrameTemplate.js
>      |   22    16 +    6 -     0 !
>
> isHostedMode(): Cute.
>

Sorry!  I'd prefer not-cute if I could make it as efficient.


>  dev/core/src/com/google/gwt/dev/GWTCompiler.java
>      |   24    21 +    3 -     0 !
>  dev/core/src/com/google/gwt/dev/GWTShell.java
>      |   24    21 +    3 -     0 !
>
> Use JavaDoc @deprecated and add a WARN message to indicate that these
> entrypoints will be removed.
>

Done, but using stderr.


>
>  dev/core/src/com/google/gwt/dev/HostedMode.java
>      |  168    120 +   48 -    0 !
>
> link() and relink() have the same JavaDoc comments.
> Suggest moving the computation of moduleOutDir and moduleExtraDir into
> a utility function, since this gets repeated in several places.
>

Fixed; added TODOs to do this later.


>  dev/core/src/com/google/gwt/dev/HostedModeBase.java
>      |   83    57 +    26 -    0 !
>
> Document refresh param on loadModule()
>

Done.


>  dev/core/src/com/google/gwt/dev/shell/BrowserWidget.java
>      |   72    39 +    33 -    0 !
>
> Update EXPECTED_GWT_ONLOAD_VERSION?
>

Good point, fixed.

Thanks!!!!!
Scott

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

Reply via email to