sergiotarxz/nuttx-apps
sergiotarxz/nuttx-apps
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

canutils: odb_decodepid: fix two debug prints

Browse Source 
Fix the following printfs, as reported by cppcheck:

canutils/libobd2/obd_decodepid.c:85:9: error: printf format string requires 1 parameter but only 0 are given. [wrongPrintfScanfArgNum]
        printf("Supported PIDs: %08X\n");
        ^
canutils/libobd2/obd_decodepid.c:114:9: error: printf format string requires 2 parameters but only 1 is given. [wrongPrintfScanfArgNum]
        printf("Throttle position = %d\% \n", (100 * dev->data[3])/255);
        ^

Signed-off-by: Eero Nurkkala <eero.nurkkala@offcode.fi>
This commit is contained in:
Eero Nurkkala 2021-12-27 14:50:50 +02:00 committed by Xiang Xiao
parent 5ac15130db
commit 2f44643704
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
canutils/libobd2/obd_decodepid.c
View File

@ -82,7 +82,7 @@ FAR char *obd_decode_pid(FAR struct obd_dev_s *dev, uint8_t pid)
(dev->data[5] << 8) | dev->data[6];
snprintf(g_data, MAXDATA, "%08X", pids);
#ifdef CONFIG_DEBUG_INFO
printf("Supported PIDs: %08X\n");
printf("Supported PIDs: %08X\n", pids);
#endif
break;
@ -111,7 +111,7 @@ FAR char *obd_decode_pid(FAR struct obd_dev_s *dev, uint8_t pid)
case OBD_PID_THROTTLE_POSITION:
snprintf(g_data, MAXDATA, "%d", (100 * dev->data[3])/255);
#ifdef CONFIG_DEBUG_INFO
printf("Throttle position = %d\%\n", (100 * dev->data[3])/255);
printf("Throttle position = %d\n", (100 * dev->data[3]) / 255);
#endif
break;
}