-
Notifications
You must be signed in to change notification settings - Fork 177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Undoing searches #376
Comments
This could be done, but you'd have to redo the search due to the possibility of the matches being moved around in memory. If I remember correctly, someone has requested something involving saving matches and @sriemer said it's unfeasible ATM. |
I was simply thinking about saving the old matches array instead of replacing it with the new one. Of course stuff may have moved in the meantime, but that's an issue with any consecutive scan, irrespective of undo. Did I miss something? |
#341 same question right? Personally I like the idea, if you have enough memory, it'd be convenient. We could add an option to automatically prune the addresses that don't match the search in the old matches. |
Yeah, it's the same thing. The OP of this issue suggested I could add a configurable number of arrays to retain, so we can have it at0 by default (= old behaviour) but let front ends move it to 2 or 10 or 3000, let them or the user handle the decision of how much history to keep. Of course the more you wait before using old data, the more and more it becomes garbage, that's just how it is... |
Sounds good to me. Perhaps we could check memory usage and show a warning if the user starts using enough RAM to slow down their system; With today's RAM, only with a high N would that be likely, however. Maybe we could even generalize this warning, it seems a lot of people have blown their stack. |
If swap is enabled on their system it should be still okay, but I noticed it does indeed completely crash the system when there isn't any ram and swap memory left still. For the way to store them in my opinion it would be nice to do unscans as much as we want so we can get back in the first search. |
I was talking with @12345ieee about adding an option for undoing to scanmem. Something like "undo_enabled" with a limit you could set e.g. "undo_limit"
For example, if you looked for a 1 in the program after which you look for 4. Searching for the 4 was a mistake so you'd want to go back to the 1, so you'd do the undo. I don't think needing to support more than the 1 undo is necessary.
The text was updated successfully, but these errors were encountered: