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 /libbb/read.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 'libbb/read.c')
-rw-r--r-- | libbb/read.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libbb/read.c b/libbb/read.c index 288358d79..fb903c18a 100644 --- a/libbb/read.c +++ b/libbb/read.c | |||
@@ -111,7 +111,7 @@ void xread(int fd, void *buf, size_t count) | |||
111 | { | 111 | { |
112 | if (count) { | 112 | if (count) { |
113 | ssize_t size = full_read(fd, buf, count); | 113 | ssize_t size = full_read(fd, buf, count); |
114 | if (size != count) | 114 | if ((size_t)size != count) |
115 | bb_error_msg_and_die("short read"); | 115 | bb_error_msg_and_die("short read"); |
116 | } | 116 | } |
117 | } | 117 | } |
@@ -160,7 +160,7 @@ char *xmalloc_reads(int fd, char *buf, size_t *maxsz_p) | |||
160 | 160 | ||
161 | goto jump_in; | 161 | goto jump_in; |
162 | while (sz < maxsz) { | 162 | while (sz < maxsz) { |
163 | if (p - buf == sz) { | 163 | if ((size_t)(p - buf) == sz) { |
164 | jump_in: | 164 | jump_in: |
165 | buf = xrealloc(buf, sz + 128); | 165 | buf = xrealloc(buf, sz + 128); |
166 | p = buf + sz; | 166 | p = buf + sz; |