Skip to content

An unread memory file keeps its unflushed flag, is never overwritten, and is named when the index refuses - #111

Merged
Coder8124 merged 3 commits into
mainfrom
unread-memfile-holes
Nov 3, 2026
Merged

Coder8124 merged 3 commits into
mainfrom
unread-memfile-holes

Conversation

@Coder8124

Copy link
Copy Markdown
Owner

Stacked on #109. Three holes in its #369 fix, found by review of #110, which this replaces:

  • An unread memory file (unreadable, or with no memory line) put its row in keep, so the index cleared that row's unflushed flag. Nothing confirmed the memory was in the vault. The row is still kept from the reap, but its flag now stays set, for doctor and the next index.
  • writeMemoryFile ran its hand-edit check only if the file could be read. When it couldn't, the write went over whatever a person had put there. It now refuses and says why.
  • An index that refuses over a damaged digest returned before it printed the unread-file notice, so the run that found the file said nothing about it. It now prints the notice first.

Each test fails on #109's code (scripts/check-bug-tests.sh origin/bugfix-0.5.7-open-bugs-v2 HEAD).

@Coder8124
Coder8124 merged commit ffb3dcd into main Nov 3, 2026
12 checks passed
@Coder8124 Coder8124 mentioned this pull request Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant