Aksal/nuttx-apps
1
0
Fork 1
mirror of https://github.com/apache/nuttx-apps.git synced 2026-08-01 20:29:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

apps/system/nxplayer&nxlooper: fix codechecker warning

Browse source 
nxplayer_main.c:
code checker warning:
level_percent = (uint8_t) atoi(parg);
     'atoi' used to convert a string to an integer value, but function will not report conversion errors; consider using 'strtol' instead.
nxlooper_main.c:
code checker warning:
percent = (uint16_t)(atof(parg) * 10.0);
     'atof' used to convert a string to a floating-point value, but function will not report conversion errors; consider using 'strtod' instead

Signed-off-by: liubojun1 <liubojun1@xiaomi.com>
This commit is contained in:
liubojun1 2022-08-31 18:07:22 +08:00 • committed by Petro Karashchenko
parent 1bd7554f5c
commit 928fb2be6d
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
system/nxlooper/nxlooper_main.c
View file

@ -264,7 +264,7 @@ static int nxlooper_cmd_volume(FAR struct nxlooper_s *plooper, char *parg)
{
/* Get the percentage value from the argument */
percent = (uint16_t)(atof(parg) * 10.0);
percent = (uint16_t)(strtof(parg, NULL) * 10.0f);
return nxlooper_setvolume(plooper, percent);
}

6
system/nxplayer/nxplayer_main.c
View file

@ -415,7 +415,7 @@ static int nxplayer_cmd_bass(FAR struct nxplayer_s *pplayer, char *parg)
{
/* Get the level and range percentage value from the argument */
level_percent = (uint8_t) atoi(parg);
level_percent = (uint8_t)strtoul(parg, NULL, 10);
nxplayer_setbass(pplayer, level_percent);
}
@ -445,7 +445,7 @@ static int nxplayer_cmd_treble(FAR struct nxplayer_s *pplayer, char *parg)
{
/* Get the level and range percentage value from the argument */
level_percent = (uint8_t) atoi(parg);
level_percent = (uint8_t)strtoul(parg, NULL, 10);
nxplayer_settreble(pplayer, level_percent);
}
@ -476,7 +476,7 @@ static int nxplayer_cmd_balance(FAR struct nxplayer_s *pplayer, char *parg)
{
/* Get the percentage value from the argument */
percent = (uint16_t) (atof(parg) * 10.0);
percent = (uint16_t)(strtof(parg, NULL) * 10.0f);
nxplayer_setbalance(pplayer, percent);
}