Skip to content

type 'NodeList' is not a subtype of type 'List<Element>' of 'directChildren' when debugging in IntelliJ #24868

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
MikeMitterer opened this issue Nov 10, 2015 · 8 comments
Assignees

Comments

@MikeMitterer
Copy link

This line: https://goo.gl/UjFEkY produces the above error message but only while debugging this module in IntelliJ 15
screenshot-1922

It works in all other cases - Chromium/Dartium VM + dar2js Version

@alan-knight
Copy link
Contributor

Are you running it in checked mode in any of the other cases? That message seems correct, in that NodeList is not a List < Element > , it implements List < Node > . And the return type of childNodes is List < Node > , so assigning it to List < Element > is not valid.

@mit-mit mit-mit added the legacy-area-analyzer Use area-devexp instead. label Nov 11, 2015
@bwilkerson bwilkerson added area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. and removed legacy-area-analyzer Use area-devexp instead. labels Nov 16, 2015
@iposva-google iposva-google removed the area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. label Nov 16, 2015
@iposva-google
Copy link
Contributor

As @MikeMitterer said this properly works on Dartium/VM unless used by IntelliJ 15. Not sure what IntelliJ's debugger is doing in this case. Eric?

@mit-mit mit-mit added the legacy-area-analyzer Use area-devexp instead. label Nov 18, 2015
@mit-mit
Copy link
Member

mit-mit commented Nov 18, 2015

@clayberg can you comment on Ivan's question above?

@bwilkerson
Copy link
Member

The error is occurring at run-time, not during analysis.

@bwilkerson bwilkerson removed the legacy-area-analyzer Use area-devexp instead. label Nov 19, 2015
@clayberg clayberg assigned jwren and unassigned clayberg Nov 19, 2015
@jwren
Copy link
Member

jwren commented Nov 19, 2015

Which version of Dartium/ IntelliJ are you using? Will go look at the flags that we are passing in (not sure of even a flag that exists that enables such behavior.)

  • Jaime

@MikeMitterer
Copy link
Author

Im using
IntelliJ IDEA (Minerva) IU-143.870.1
Build #IU-143.870, built on November 18, 2015
JRE: 1.8.0_40-release-b92 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

and Chromium Version 39.0.2171.0
Since dart 1.13 this is also a problem in Dart2JS - its ok that this is an error but dartanalyzer should warn me about this and not just crash in test or in dart2js.

@kasperl kasperl added the legacy-area-analyzer Use area-devexp instead. label Nov 27, 2015
@bwilkerson
Copy link
Member

@kasperl Not sure why you added the "area-analyzer" label to this issue. It appears to me to be an interaction between IntelliJ and Dartium, having nothing to do with the analyzer. I think we should either move it out of the 'sdk' issue tracker or create a new "area-IntelliJ" label for such issues.

@bwilkerson bwilkerson removed the legacy-area-analyzer Use area-devexp instead. label Dec 15, 2015
@srawlins srawlins changed the title type 'NodeList' is not a subtype of type 'List<Element>' of 'directChildren' when debugging in ItelliJ type 'NodeList' is not a subtype of type 'List<Element>' of 'directChildren' when debugging in IntelliJ Apr 25, 2017
@terrylucas
Copy link
Contributor

Please re-test with Dart 1.24.2 Stable (Dartium 50).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants