diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-05-13 02:27:31 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-05-13 02:27:31 +0000 |
commit | 77ad97f199f1bf05e9a7609bbdd239dab825b258 (patch) | |
tree | cf117ebf8d4a50bc7ba0e4da4d60a98a944756c8 /coreutils/dd.c | |
parent | c4f12f59cc907577d787f816b37122809f896bb2 (diff) | |
download | busybox-w32-77ad97f199f1bf05e9a7609bbdd239dab825b258.tar.gz busybox-w32-77ad97f199f1bf05e9a7609bbdd239dab825b258.tar.bz2 busybox-w32-77ad97f199f1bf05e9a7609bbdd239dab825b258.zip |
more -Wall warning fixes from Cristian Ionescu-Idbohrn.
This time it resulted in small code changes:
function old new delta
nexpr 820 828 +8
tail_main 1200 1202 +2
wrapf 166 167 +1
parse_mount_options 227 209 -18
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 3/1 up/down: 11/-18) Total: -7 bytes
Diffstat (limited to 'coreutils/dd.c')
-rw-r--r-- | coreutils/dd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/coreutils/dd.c b/coreutils/dd.c index 4d1ef0b76..6b66366b6 100644 --- a/coreutils/dd.c +++ b/coreutils/dd.c | |||
@@ -65,7 +65,7 @@ static bool write_and_stats(const void *buf, size_t len, size_t obs, | |||
65 | ssize_t n = full_write_or_warn(buf, len, filename); | 65 | ssize_t n = full_write_or_warn(buf, len, filename); |
66 | if (n < 0) | 66 | if (n < 0) |
67 | return 1; | 67 | return 1; |
68 | if (n == obs) | 68 | if ((size_t)n == obs) |
69 | G.out_full++; | 69 | G.out_full++; |
70 | else if (n) /* > 0 */ | 70 | else if (n) /* > 0 */ |
71 | G.out_part++; | 71 | G.out_part++; |
@@ -312,7 +312,7 @@ int dd_main(int argc ATTRIBUTE_UNUSED, char **argv) | |||
312 | while (n) { | 312 | while (n) { |
313 | size_t d = obs - oc; | 313 | size_t d = obs - oc; |
314 | 314 | ||
315 | if (d > n) | 315 | if (d > (size_t)n) |
316 | d = n; | 316 | d = n; |
317 | memcpy(obuf + oc, tmp, d); | 317 | memcpy(obuf + oc, tmp, d); |
318 | n -= d; | 318 | n -= d; |