flutter / flutter/devtools

`dcm check-unused-code`: bugs discovered where code is incorrectly unused.

Aperta
#9,910 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug cleanup P2
Lingua principale
Dart
Stelle
1.7k
Fork
404
Merge medio
6g 17h
PR unite (30g)
18

Descrizione

Findings:
1. **vm_developer** screen: this to delete the unused code requires deleting most of this file. It seems like a bug that the description is not used. CC @bkonyi
```
packages/devtools_app/lib/src/screens/vm_developer/object_inspector/inbound_references_tree.dart (1 issue):
✖ unused field description
at packages/devtools_app/lib/src/screens/vm_developer/object_inspector/inbound_references_tree.dart:29:3
```
2. **vm_developer** screen: it seems like a bug if we are intending `ProfileCode` to be serializable but it is unused. CC @bkonyi
```
packages/devtools_app/lib/src/screens/vm_developer/vm_service_private_extensions.dart (8 issues):
✖ unused method toJson
at packages/devtools_app/lib/src/screens/vm_developer/vm_service_private_extensions.dart:856:3
```
3. **profiler** screen: we used to have functionality to load an app startup profile by a user tag. This should be investigated to see if this method can be removed or if the fact that this is unused is a regression.
```
packages/devtools_app/lib/src/screens/profiler/cpu_profiler_controller.dart (1 issue):
✖ unused method loadAppStartUpProfile
at packages/devtools_app/lib/src/screens/profiler/cpu_profiler_controller.dart:414:3
```
4. **app size screen**
```
packages/devtools_app/lib/src/shared/charts/treemap.dart (1 issue):
✖ unused field childrenMap
at packages/devtools_app/lib/src/shared/charts/treemap.dart:685:3
```

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia eseguendo `dcm check-unused-code` e ispeziona le posizioni segnalate in `inbound_references_tree.dart`, `vm_service_private_extensions.dart`, `cpu_profiler_controller.dart` e `treemap.dart`. Determina se ogni risultato è realmente un membro non utilizzato, un percorso di serializzazione previsto o una regressione nel caricamento del profilo di avvio dell’app; il lavoro è completato quando ogni risultato ha una rimozione o una correzione motivata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
dart
Ambito
devtools, tooling
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.