diff options
author | Richard Fitzgerald <[email protected]> | 2021-05-25 13:20:11 +0100 |
---|---|---|
committer | Petr Mladek <[email protected]> | 2021-05-27 15:38:03 +0200 |
commit | 1b932689c77766b68e2ead51ca0fb84ec5bb8965 (patch) | |
tree | 21122208e0a00f3c39c351bbf8d6b0df2fe6b853 /lib/vsprintf.c | |
parent | ef04d4ff4b19628c78abddc768acce097d35d086 (diff) |
lib: test_scanf: Remove pointless use of type_min() with unsigned types
sparse was producing warnings of the form:
sparse: cast truncates bits from constant value (ffff0001 becomes 1)
There is no actual problem here. Using type_min() on an unsigned type
results in an (expected) truncation.
However, there is no need to test an unsigned value against type_min().
The minimum value of an unsigned is obviously 0, and any value cast to
an unsigned type is >= 0, so for unsigneds only type_max() need be tested.
This patch also takes the opportunity to clean up the implementation of
simple_numbers_loop() to use a common pattern for the positive and
negative test.
Reported-by: kernel test robot <[email protected]>
Signed-off-by: Richard Fitzgerald <[email protected]>
Reviewed-by: Petr Mladek <[email protected]>
Signed-off-by: Petr Mladek <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'lib/vsprintf.c')
0 files changed, 0 insertions, 0 deletions