snprintf uses %d on a double (issue 2784)

smoe · andypugh · commit 86ce4bb72713 · 2023-12-14T13:38:51.000Z
diff --git a/src/emc/usr_intf/emcrsh.cc b/src/emc/usr_intf/emcrsh.cc
@@ -1713,7 +1713,7 @@ static cmdResponseType getTool(char *s, connectionRecType *context)
 
 static cmdResponseType getToolOffset(char *s, connectionRecType *context)
 {
-  const char *pToolOffsetStr = "TOOL_OFFSET %d";
+  const char *pToolOffsetStr = "TOOL_OFFSET %f";
   
   snprintf(context->outBuf, sizeof(context->outBuf), pToolOffsetStr, emcStatus->task.toolOffset.tran.z);
   return rtNoError; 

Original file line number	Diff line number	Diff line change
`@@ -1713,7 +1713,7 @@ static cmdResponseType getTool(char s, connectionRecType context)`
`1713`	`1713`
`1714`	`1714`	`static cmdResponseType getToolOffset(char s, connectionRecType context)`
`1715`	`1715`	`{`
`1716`		`- const char *pToolOffsetStr = "TOOL_OFFSET %d";`
	`1716`	`+ const char *pToolOffsetStr = "TOOL_OFFSET %f";`
`1717`	`1717`
`1718`	`1718`	`snprintf(context->outBuf, sizeof(context->outBuf), pToolOffsetStr, emcStatus->task.toolOffset.tran.z);`
`1719`	`1719`	`return rtNoError;`