Add "go to previous level" key to replay viewer

[tmewett]

There is a "go to next level," so I feel there should be a go to previous, which goes back to the start of the previous depth. This would have to be implemented like any rewinding operation currently, i.e. with a confirm prompt and by going to the start and fast-forwarding.

[kz-dev]

I will start to check this issue !

So I found AdvanceToLocation who take a turn. So we could store all first turn of each level and used them when we press "<" with a confirm prompt.

What do you think about this technique ?

[tmewett]

I think storage of first turns on each level may not be necessary. How does the implementation of go to next level ('>') work? I think we might be able to define '<' as something like

1. save (current depth - 1) to N
2. rewind to start of recording
3. do '>' max{0, N-1} times

[kz-dev]

">" implementation is just a "while" until we are to the next level (with some option to fastforwarding) :

pauseState = rogue.playbackPaused; previousDeepestLevel = rogue.deepestLevel; if (!rogue.playbackPaused || unpause()) { if ((unsigned long) rogue.deepestLevel < maxLevelChanges) { displayCenteredAlert(" Loading... "); commitDraws(); rogue.playbackFastForward = true; while ((rogue.deepestLevel <= previousDeepestLevel || !rogue.playbackBetweenTurns) && !rogue.gameHasEnded) { rogue.RNG = RNG_COSMETIC; // dancing terrain colors can't influence recordings nextBrogueEvent(&theEvent, false, true, false); rogue.RNG = RNG_SUBSTANTIVE; executeEvent(&theEvent); } rogue.playbackFastForward = false; rogue.playbackPaused = pauseState; displayLevel(); refreshSideBar(-1, -1, false); updateMessageDisplay(); } else { flashTemporaryAlert(" Already reached deepest depth explored ", 1000); } } rogue.playbackPaused = pauseState;

And the function

BrogueCE/src/brogue/Recordings.c

Line 612 in f3bb9a5

void advanceToLocation(unsigned long destinationFrame) {

allow us to go at a turn number.

Can we standardize "<" and ">" function with this function (advanceToLocation) ?

• We just need to store an array with the turn number of each start level when we read the recording file (parseFile)
• When < or > press key : Navigate in the array with advanceToLocation

It's just an idea, else I can implement your previous suggestion.

[tmewett]

When the target location is in the past, advanceToLocation works in a similar way: by rewinding to the start and fast-forwarding. Because of this, storing any data is unnecessary - instead of fast-forwarding until we hit a certain turn number (advanceToLocation), we just fast-forward until we get to a certain depth. Does that make sense?

[kz-dev]

Yes, my reflexion was a bit stupid... I will propose you something !