Reducing Selection to only rely on nodeIndexCalculator and not on nodeIndex
- Vorherrschende Sprache
- Dart
- Sterne
- 1.7k
- Forks
- 404
- Ø Merge
- 6 T. 17 Std.
- Gemergte PRs (30 T.)
- 18
Beschreibung
**Currently**
Selection class in table.dart contains both nodeIndex and nodeIndexCalculator. https://github.com/flutter/devtools/blob/master/packages/devtools_app/lib/src/shared/table.dart#L321
**Future**
Only NodeIndexCalculator is needed and all changes to nodeIndex can be migrated to nodeIndexCalculator. Only changes of nodeIndex is found in memory_heap_tree.
Beitragsleitfaden
Rechercherichtung
Beginne in packages/devtools_app/lib/src/shared/table.dart, wo Selection derzeit nodeIndex und nodeIndexCalculator enthält. Untersuche anschließend memory_heap_tree auf jede Änderung von nodeIndex und verfolge, wie diese Aktualisierungen zu nodeIndexCalculator verschoben werden können. Erledigt ist es, wenn Selection nicht mehr auf nodeIndex angewiesen ist und der memory heap tree den Calculator konsistent verwendet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- dart
- Bereich
- devtools
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 52/100