Context Navigation

← Previous Change
Ticket Comment History
Next Change →

Changes between Initial Version and Version 1 of Ticket #622, comment 3

Timestamp:: 05/16/19 08:24:42 (6 years ago)
Author:: Alpar Juttner
Comment:

Legend:

: Unmodified
: Added
: Removed
: Modified

Ticket #622, comment 3

-                      initial
+                      v1
 Syntactically, you could indeed replace `std::vector<Item> _items` with `Item _item` plus every instance of `_items[0]` with `_item`, and it would compile beautifully. But would crash immediately when trying to use it. Because the ''whole'' vector is indeed in heavy use through pointers.
 In fact, it is a fundamental part of the data structure. It contains ''all'' items ordered by their levels and by there activeness. I.e. it stars with the active items on level 0, then the inactive ones on level 0, then the active ones on level 1, then the inactive ones on level 1 etc.
 The vectors `_first` and `_last_active` indicates the borders of these intervals on each level. In addition, the map `_where` gives a cross reference to the elements. The values of these vectors are ''pointers'' (`Item *`), and the elements of `_items` are accessed and manipulated through them.
+In fact, it is a fundamental part of the data structure. It contains ''all'' items ordered by their levels and by their activeness. I.e. it starts with the active items on level 0, then come the inactive ones on level 0, then the active ones on level 1, then the inactive ones on level 1 etc.
+The vectors `_first` and `_last_active` indicate the borders of these intervals on each level. In addition, the map `_where` gives a cross reference to the elements. The values of these vectors are ''pointers'' (`Item *`), and the elements of `_items` are accessed and manipulated through them.
 Right at the beginning, `initStart()` puts ''all'' items into `_item`. Later on, its values are manipulated with the functions `copy()` and `swap()`.