emcTaskStateRestore: return a useful(?) result

jepler · jepler · commit 64d87d2ef29f · 2022-08-21T10:39:58.000-05:00
.. it's ignored in the caller, but this should be the return value.
It quiets a diagnostic about res being assigned but not used.
diff --git a/src/emc/task/emctask.cc b/src/emc/task/emctask.cc
@@ -217,13 +217,13 @@ int emcTaskHalt()
 
 int emcTaskStateRestore()
 {
-    int res;
+    int res = 0;
     // Do NOT restore on MDI command
     if (emcStatus->task.mode == EMC_TASK_MODE_AUTO) {
         // Validity of state tag checked within restore function
         res = pinterp->restore_from_tag(emcStatus->motion.traj.tag);
     }
-    return 0;
+    return res;
 }
 
 int emcTaskAbort()

Original file line number	Diff line number	Diff line change
`@@ -217,13 +217,13 @@ int emcTaskHalt()`
`217`	`217`
`218`	`218`	`int emcTaskStateRestore()`
`219`	`219`	`{`
`220`		`- int res;`
	`220`	`+ int res = 0;`
`221`	`221`	`// Do NOT restore on MDI command`
`222`	`222`	`if (emcStatus->task.mode == EMC_TASK_MODE_AUTO) {`
`223`	`223`	`// Validity of state tag checked within restore function`
`224`	`224`	`res = pinterp->restore_from_tag(emcStatus->motion.traj.tag);`
`225`	`225`	`}`
`226`		`- return 0;`
	`226`	`+ return res;`
`227`	`227`	`}`
`228`	`228`
`229`	`229`	`int emcTaskAbort()`