Skip to content

Commit b45e7ab

Browse files
committed
Args.c: remove variable validation
we already know the exact value of the name variable so there's no need to validate it.
1 parent 9268fba commit b45e7ab

File tree

1 file changed

+7
-7
lines changed
  • nx-X11/programs/Xserver/hw/nxagent

1 file changed

+7
-7
lines changed

nx-X11/programs/Xserver/hw/nxagent/Args.c

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1406,7 +1406,7 @@ static void nxagentParseOptions(char *name, char *value)
14061406
{
14071407
fprintf(stderr, "Warning: Unable to convert value [%s] of option [%s]. "
14081408
"Ignoring option.\n",
1409-
validateString(value), validateString(name));
1409+
validateString(value), name);
14101410

14111411
return;
14121412
}
@@ -1417,7 +1417,7 @@ static void nxagentParseOptions(char *name, char *value)
14171417

14181418
fprintf(stderr, "Warning: value [%s] of option [%s] "
14191419
"out of range, clamped to [%lu].\n",
1420-
validateString(value), validateString(name), sleep_parse);
1420+
validateString(value), name, sleep_parse);
14211421
}
14221422

14231423
if (0 > sleep_parse)
@@ -1426,7 +1426,7 @@ static void nxagentParseOptions(char *name, char *value)
14261426

14271427
fprintf(stderr, "Warning: value [%s] of option [%s] "
14281428
"out of range, clamped to [%lu].\n",
1429-
validateString(value), validateString(name), sleep_parse);
1429+
validateString(value), name, sleep_parse);
14301430
}
14311431

14321432
nxagentChangeOption(SleepTime, sleep_parse);
@@ -1465,7 +1465,7 @@ static void nxagentParseOptions(char *name, char *value)
14651465
{
14661466
fprintf(stderr, "Warning: Unable to convert value [%s] of option [%s]. "
14671467
"Ignoring option.\n",
1468-
validateString(value), validateString(name));
1468+
validateString(value), name);
14691469

14701470
return;
14711471
}
@@ -1476,7 +1476,7 @@ static void nxagentParseOptions(char *name, char *value)
14761476

14771477
fprintf(stderr, "Warning: value [%s] of option [%s] "
14781478
"out of range, clamped to [%lu].\n",
1479-
validateString(value), validateString(name), tolerance_parse);
1479+
validateString(value), name, tolerance_parse);
14801480
}
14811481

14821482
if (0 > tolerance_parse)
@@ -1485,7 +1485,7 @@ static void nxagentParseOptions(char *name, char *value)
14851485

14861486
fprintf(stderr, "Warning: value [%s] of option [%s] "
14871487
"out of range, clamped to [%lu].\n",
1488-
validateString(value), validateString(name), tolerance_parse);
1488+
validateString(value), name, tolerance_parse);
14891489
}
14901490

14911491
#ifdef TEST
@@ -1499,7 +1499,7 @@ static void nxagentParseOptions(char *name, char *value)
14991499
fprintf(stderr, "nxagentParseOptions: Warning: value [%s] of "
15001500
"option [%s] unknown, will be mapped to "
15011501
"\"Bypass\" [%u] value internally.\n",
1502-
validateString(value), validateString(name),
1502+
validateString(value), name,
15031503
(unsigned int)ToleranceChecksBypass);
15041504
}
15051505
#endif

0 commit comments

Comments
 (0)