Giving up on the change to report errors with factories and generic types.

Giving up on the change to report errors with factories and generic types.
This means that you will continue to have crashes in Debug mode, but you will be able to "run" in Release mode. Best solution is to get rid of generic types in Link until factories and generic types have been fully implemented. Until then we do not know what can happen when running in release mode.

Still trying to figure out how to test frog and leg correctly.
Committed: https://code.google.com/p/dart/source/detail?r=1469

[regis, 2011-11-11 18:47:13]

LGTM

http://codereview.chromium.org/8540013/diff/2002/frog/leg/util/link_implement...
File frog/leg/util/link_implementation.dart (left):

http://codereview.chromium.org/8540013/diff/2002/frog/leg/util/link_implement...
frog/leg/util/link_implementation.dart:8: class LinkFactory<T> {
As a temporary workaround, you could try to have class LinkFactory<T> implements
Link<T> { ... }

http://codereview.chromium.org/8540013/diff/2002/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8540013/diff/2002/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:6392: void Parser::CheckConstructorCallTypeArguments(
How about removing this function alltogether? It will not be used once factories
are implemented correctly.

[srdjan, 2011-11-11 18:49:19]

http://codereview.chromium.org/8540013/diff/2002/frog/leg/util/link_implement...
File frog/leg/util/link_implementation.dart (left):

http://codereview.chromium.org/8540013/diff/2002/frog/leg/util/link_implement...
frog/leg/util/link_implementation.dart:8: class LinkFactory<T> {
On 2011/11/11 18:47:13, regis wrote:
> As a temporary workaround, you could try to have class LinkFactory<T>
implements
> Link<T> { ... }

I try this next once, I have the build bot green again

http://codereview.chromium.org/8540013/diff/2002/runtime/vm/parser.cc
File runtime/vm/parser.cc (right):

http://codereview.chromium.org/8540013/diff/2002/runtime/vm/parser.cc#newcode...
runtime/vm/parser.cc:6392: void Parser::CheckConstructorCallTypeArguments(
On 2011/11/11 18:47:13, regis wrote:
> How about removing this function alltogether? It will not be used once
factories
> are implemented correctly.

I try the fix you suggested (implement) and reenable this if it helps.