Review fixes

src/BookmarkStore.h

@@ -83,7 +83,7 @@ class BookmarkStore {
       return;
     }
 
-    if (bookmarks.size() > UINT16_MAX) {
+    if (bookmarks.size() > MAX_BOOKMARKS) {
       LOG_ERR("BKM", "Too many bookmarks to save: %u", static_cast<unsigned>(bookmarks.size()));
       return;
     }
@@ -109,7 +109,6 @@ class BookmarkStore {
       }
     }
 
-    bool closeOk = false;
     if (ok) {
       if (!f.close()) {
         LOG_ERR("BKM", "Failed to close bookmarks file");

src/GlobalBookmarkIndex.cpp

@@ -176,7 +176,11 @@ bool GlobalBookmarkIndex::reconcile() {
   entries.erase(std::remove_if(entries.begin(), entries.end(),
                                [](const Entry& e) {
                                  if (!Storage.exists(e.sourcePath.c_str())) {
-                                   LOG_DBG("GBI", "Dropping orphan entry: %s", e.sourcePath.c_str());
+                                   LOG_DBG("GBI", "Dropping orphan entry: missing sourcePath=%s", e.sourcePath.c_str());
+                                   return true;
+                                 }
+                                 if (!Storage.exists(e.cacheDir.c_str())) {
+                                   LOG_DBG("GBI", "Dropping orphan entry: missing cacheDir=%s", e.cacheDir.c_str());
                                    return true;
                                  }
                                  return false;

src/activities/home/GlobalBookmarksActivity.cpp

@@ -20,9 +20,7 @@
 void GlobalBookmarksActivity::onEnter() {
   Activity::onEnter();
 
-  if (GLOBAL_BOOKMARKS.reconcile()) {
+  GLOBAL_BOOKMARKS.reconcile();
-    GLOBAL_BOOKMARKS.save();
-  }
   rebuildRows();
 
   const int first = firstSelectableIndex();

src/activities/home/HomeActivity.cpp

@@ -260,7 +260,7 @@ void HomeActivity::loop() {
   if (firstRenderDone && !recentsLoaded && !recentsLoading) {
     const auto& metrics = UITheme::getInstance().getMetrics();
     const Rect contentRect = UITheme::getContentRect(renderer, true, false);
-    const int menuItemCount = hasOpdsUrl ? 6 : 5;
+    const int menuItemCount = getMenuItemCount();
     const HomeScreenLayout layout = computeHomeScreenLayout(metrics, contentRect.height, menuItemCount);
     loadRecentCovers(getHomeCoverRenderHeight(layout));
     return;

src/activities/reader/TxtReaderActivity.cpp

@@ -438,6 +438,8 @@ void TxtReaderActivity::applyPendingBookmarkJump() {
     return;
   }
   LOG_DBG("TRS", "Applying pending bookmark jump: page=%u", jump.pageNumber);
+
+  bool persisted = false;
   FsFile f;
   if (Storage.openFileForWrite("TRS", txt->getCachePath() + "/progress.bin", f)) {
     uint8_t data[6] = {0};
@@ -445,11 +447,17 @@ void TxtReaderActivity::applyPendingBookmarkJump() {
     data[1] = (jump.pageNumber >> 8) & 0xFF;
     // Offset bytes stay 0: loadProgress reads only the page, and the lazy
     // initializeReader() rebuilds the page index on first render anyway.
-    f.write(data, 6);
+    if (f.write(data, 6) == 6) {
-    f.close();
+      persisted = f.close();
+    } else {
+      f.close();
+    }
+  }
+
+  if (persisted) {
+    jump.clear();
+    APP_STATE.saveToFile();
   }
-  jump.clear();
-  APP_STATE.saveToFile();
 }
 
 void TxtReaderActivity::loadProgress() {