From 1ffe7367a3ad468f00aec19d887dcb295bee4edb Mon Sep 17 00:00:00 2001
From: James Crook <james.k.crook@gmail.com>
Date: Fri, 24 Feb 2017 17:21:17 +0000
Subject: [PATCH] Comment about missing/broken ERR_HANDLING.

---
 src/DirManager.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/DirManager.cpp b/src/DirManager.cpp
index aa834b96e..5652b0cc6 100644
--- a/src/DirManager.cpp
+++ b/src/DirManager.cpp
@@ -514,6 +514,12 @@ bool DirManager::SetProject(wxString& newProjPath, wxString& newProjName, const
       {
          BlockFilePtr b = iter->second.lock();
          if (b) {
+            // FIXME: TRAP_ERR
+            // JKC: The 'success' variable and recovery strategy looks 
+            // broken/bogus to me.  Would need to be using &= to catch 
+            // failure in one of the copies/moves.  Besides which,
+            // our temporary files are going to be deleted when we exit 
+            // anyway, if saving from temporary to named project.
             if (b->IsLocked())
                success = CopyToNewProjectDirectory( &*b ), copying = true;
             else{