Thomas Harte
|
5d63556870
|
Periods need a custom copy constructor too, if they're going to avoid sharing an event_source.
|
2016-12-30 17:39:52 -05:00 |
|
Thomas Harte
|
63ff5165a4
|
After a quick bit of reading, discovered the virtual copy constructor pattern really is only a convention in C++, and conformed to it. Which hopefully gives copyable tracks.
|
2016-12-30 17:25:39 -05:00 |
|
Thomas Harte
|
af1b396c9e
|
Found an ugly issue with Storage::Time as implemented (i) to be unsigned; and (ii) automatically to simplify. Will need to fix. Here's a quick workaround for this one segment of code.
|
2016-12-28 22:57:11 -05:00 |
|
Thomas Harte
|
742c5df367
|
With lots of logging arising temporarily, fixed bug whereby conversion to a patched track would lead to holding a track with a distinct measure of time, leading to improperly-placed patches.
|
2016-12-25 22:00:39 -05:00 |
|
Thomas Harte
|
b538ee5bd8
|
Fixed discovery of correct active period and setting of track time, when seeking.
|
2016-12-25 21:32:50 -05:00 |
|
Thomas Harte
|
03579f33f1
|
Fixed multi-coverage insertion, via an appropriate test.
|
2016-12-20 21:38:32 -05:00 |
|
Thomas Harte
|
7eca910cc5
|
Fixed insertion location finding logic, working on the relevant test.
|
2016-12-20 21:14:05 -05:00 |
|
Thomas Harte
|
823ab9bc34
|
Completed initial non-trivial test, fixing revealed errors.
|
2016-12-20 19:15:36 -05:00 |
|
Thomas Harte
|
5a508ea0df
|
Attempted properly to cover the exactly-equal starts and ends cases, and to improve meaning.
|
2016-12-20 18:32:49 -05:00 |
|
Thomas Harte
|
63d861a2f3
|
Switched from C-in-the-brain manual offset counting to using iterators like an ordinary C++ person.
|
2016-12-20 18:17:54 -05:00 |
|
Thomas Harte
|
6f17076003
|
Switched to much more logical shared_ptr ownership of PCMSegmentEventSources by Periods.
|
2016-12-20 18:13:10 -05:00 |
|
Thomas Harte
|
497b2ae4dd
|
Still by manual inspection: the time for the next event should be provisional until proven acceptable, allowing a proper measurement of time until exiting the period to be taken; also fixed the accumulated period error when seeking back onto the underlying track.
|
2016-12-20 08:14:16 -05:00 |
|
Thomas Harte
|
6bdde542c5
|
Edging towards functioning automatic tests, fixed right-period adjustment and slightly decreased searching cost while in the process of adding a test.
|
2016-12-20 07:52:14 -05:00 |
|
Thomas Harte
|
ec624eaab1
|
Made an attempt fully to implement PCMPatchedTrack. Which now requires tests.
|
2016-12-20 07:30:57 -05:00 |
|
Thomas Harte
|
1ef1f6ec69
|
Attempted to implemnt seek_to and to finish add_segment. Started doing a little of get_next_event but ran out of time for the day.
|
2016-12-19 21:46:02 -05:00 |
|
Thomas Harte
|
8f937ceac8
|
Made an attempt to come up with a data structure that actually makes sense (though perhaps this is textbook list rather than vector stuff? I guess it depends on the frequency I expect inserts to occur versus reads) and to implement inserts. Though the Periods aren't yet honoured.
|
2016-12-19 07:42:43 -05:00 |
|
Thomas Harte
|
313db75303
|
Ensured the patchable track owns its underlying track.
|
2016-12-17 18:17:22 -05:00 |
|
Thomas Harte
|
f1a08b7ab5
|
Opted to pass times by reference and added enough to PCMPatchedTrack that it could start being used by the disk controller, albeit that it doesn't work.
|
2016-12-17 16:26:45 -05:00 |
|
Thomas Harte
|
dc08a23ceb
|
This is going to be a slow walk, I think. This class attempts to be the scratchpad which will hold in-memory track modifications.
|
2016-12-16 19:20:38 -05:00 |
|