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 /editors/vi.c | |
| parent | c4f12f59cc907577d787f816b37122809f896bb2 (diff) | |
| download | busybox-77ad97f199f1bf05e9a7609bbdd239dab825b258.tar.gz | |
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 'editors/vi.c')
| -rw-r--r-- | editors/vi.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/editors/vi.c b/editors/vi.c index 4e5a5ac4a..5013d0d51 100644 --- a/editors/vi.c +++ b/editors/vi.c @@ -2239,7 +2239,9 @@ static char readit(void) // read (maybe cursor) key from stdin pfd[0].events = POLLIN; // keep reading while there are input chars, and room in buffer // for a complete ESC sequence (assuming 8 chars is enough) - while (safe_poll(pfd, 1, 0) > 0 && n <= (sizeof(readbuffer) - 8)) { + while ((safe_poll(pfd, 1, 0) > 0) + && ((size_t)n <= (sizeof(readbuffer) - 8)) + ) { // read the rest of the ESC string int r = safe_read(0, readbuffer + n, sizeof(readbuffer) - n); if (r > 0) |