Please also look at the compiled JS. I think the less restrictive bound will
cause the compiler to generate worse code (e.g. dynamic casts in some or all
contexts) which would make it a non-starter.
On Mon, Jun 8, 2009 at 6:22 PM, Stefan Haustein <haust...@google.com> wrote:
> Hi,
>
> I'd like to submit a patch (see below) that changes "JsArray<T extends
> JavaScriptObject>" to "JsArray<T>"
>
> Motivation:
> Support more lightweight code in places where we depend on Javascript
> anyway.
> In particular, I would like to remove the dependency on LinkedList in
> AsyncFragmentLoader in a follow-up change.
>
> I have verified that this change does not introduce any new test failures.
>
> Stefan
>
>
> Index: user/src/com/google/gwt/core/client/JsArray.java
> ===================================================================
> --- user/src/com/google/gwt/core/client/JsArray.java (revision 5522)
> +++ user/src/com/google/gwt/core/client/JsArray.java (working copy)
> @@ -34,7 +34,7 @@
> *
> * @param <T> the concrete type of object contained in this array
> */
> -public class JsArray<T extends JavaScriptObject> extends JavaScriptObject
> {
> +public class JsArray<T> extends JavaScriptObject {
>
> protected JsArray() {
> }
>
>
> --
> Stefan Haustein
> Google UK Limited
>
> Registered Office: Belgrave House, 76 Buckingham Palace Road, London SW1W
> 9TQ; Registered in England Number: 3977902
>
>
> >
>
--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---
