Cleanup: Remove unused function

- Unused since 2010, removed in accordance with Paul's wishes.

src/commands/CmdModify.cpp

@@ -137,7 +137,6 @@ int CmdModify::modifyAndUpdate (
   int count = 0;
 
   updateRecurrenceMask (after);
-  dependencyChainOnModify (before, after);
   ++count;
   feedback_affected (STRING_CMD_MODIFY_TASK, after);
   feedback_unblocked (after);
@@ -176,7 +175,6 @@ int CmdModify::modifyRecurrenceSiblings (
       Task alternate (sibling);
       sibling.modify (Task::modReplace);
       updateRecurrenceMask (sibling);
-      dependencyChainOnModify (alternate, sibling);
       ++count;
       feedback_affected (STRING_CMD_MODIFY_TASK_R, sibling);
       feedback_unblocked (sibling);
@@ -213,7 +211,6 @@ int CmdModify::modifyRecurrenceParent (
       child.modify (Task::modReplace);
       updateRecurrenceMask (child);
       context.tdb2.modify (child);
-      dependencyChainOnModify (alternate, child);
       if (context.verbose ("project") && projectChanges)
         (*projectChanges)[child.get ("project")] = onProjectChange (alternate, child);
       ++count;

src/dependency.cpp

@@ -209,45 +209,3 @@ void dependencyChainOnStart (Task& task)
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-// Iff a dependency is being removed, is there something to do.
-void dependencyChainOnModify (Task& before, Task& after)
-{
-  // TODO It is not clear that this should even happen.  TBD.
-/*
-  // Get the dependencies from before.
-  std::string depends = before.get ("depends");
-  std::vector <std::string> before_depends;
-  split (before_depends, depends, ',');
-  std::cout << "# dependencyChainOnModify before has " << before_depends.size () << "\n";
-
-  // Get the dependencies from after.
-  depends = after.get ("depends");
-  std::vector <std::string> after_depends;
-  split (after_depends, depends, ',');
-  std::cout << "# dependencyChainOnModify after has " << after_depends.size () << "\n";
-
-  // listDiff
-  std::vector <std::string> before_only;
-  std::vector <std::string> after_only;
-  listDiff (before_depends, after_depends, before_only, after_only);
-
-  // Any dependencies in before_only indicates that a dependency was removed.
-  if (before_only.size ())
-  {
-    std::cout << "# dependencyChainOnModify detected a dependency removal\n";
-
-    // before   dep:2,3
-    // after    dep:2
-    //
-    // any tasks blocked by after might should be repaired to depend on 3.
-
-    std::vector <Task> blocked;
-    dependencyGetBlocked (after, blocked);
-
-    for (auto& b : blocked)
-      std::cout << "# dependencyChainOnModify\n";
-  }
-*/
-}
-
-////////////////////////////////////////////////////////////////////////////////

src/main.h

@@ -58,7 +58,6 @@ void dependencyGetBlocking (const Task&, std::vector <Task>&);
 bool dependencyIsCircular (const Task&);
 void dependencyChainOnComplete (Task&);
 void dependencyChainOnStart (Task&);
-void dependencyChainOnModify (Task&, Task&);
 
 // feedback.cpp
 bool taskDiff (const Task&, const Task&);