sdk

sdk copied to clipboard

Missing inspections for unused class, unused public methods and properties

27

*@ph55 commented on Jun 16, 2020, 1:18 PM UTC:* Feature request / improvement. Classes missing some inspections for * Unused class itself * Unused public properties * Unused public methods...

ghost

No error in analyzer in case of inherited members conflict

5

The followinng code produces an error in CFE but no issues in analyzer ```dart class V1 { int m() => 42; } class V2 { String get m => "42";...

sgrekhov

Analyzer memory usage is much higher with a clean cache

2

Via https://github.com/dart-lang/sdk/issues/41793#issuecomment-1387269582 I found and checked out https://github.com/kodecocodes/rwf-materials.git and opened the entire directory (after running `flutter pub get` 500+ times --- I don't know if there's a helper command but...

jensjoha

[analyzer] consider a more gradual reanalysis after `analysis_options` file is changed

2

Saving `analysis_options` restarts the analysis server completely making it hard to lint analysis options in general (shows stale issues on save), but also consumes resources. You can repro this when...

incendial

[analyzer] strange `ambiguous_import` report

12

Hey! Just upgraded to Dart 3.0.0-417.4.beta and now see some strange reports for code inside the documentation comments https://github.com/dart-lang/sdk/blob/main/pkg/analyzer/lib/src/dart/ast/ast.dart#L5528 is it supposed to be this way?

incendial

Add analyzer warnings for classes holding FFI pointers to either release ownership or call the dispose method.

10

Hi to the team. Today I was writing a Flutter FFI library and it hit me: What if there was a way to annotate a class as holding native resources...

Piero512

[analyzer] Incorrect `unused_local_variable` warning.

2

`a` is used here: ```dart main() { final a = foo(null); a!; } foo(x) => x; ``` ``` $ dart analyze ~/play1/bug1c8i.dart Analyzing bug1c8i.dart... 0.6s warning • bug1c8i.dart:2:9 • The...

rakudrama

Analyzer allows re-use of part files from core libraries

2

Consider the following program: library dart.async; part 'dart:async/future.dart'; main() {} This program should be rejected, since `dart:async/future.dart` is not the URI of any file that can be validly used by...

stereotype441

Validate the conditions of config specific imports in analyzer

7

All tools restrict the conditions to `dart.library.*`, but analyzer ignores them completely. It would be nice to get a hint when there is a typo.

natebosch

See for example #35968, where it is not at all obvious from the stacktrace that the summary is empty.

stereotype441