diff options
| author | Evan Huus <eapache@gmail.com> | 2015-11-16 16:02:43 -0500 |
|---|---|---|
| committer | Evan Huus <eapache@gmail.com> | 2015-11-17 04:18:37 +0000 |
| commit | c3025544b63c456ba1da066e22c80b9018a64fbb (patch) | |
| tree | 1964c282903f6bb4c625a955b88fbd7f0f1e63c2 /ui/win32 | |
| parent | 9a7973770e19dca8ee455d60efb0b46c3bafef98 (diff) | |
| download | wireshark-c3025544b63c456ba1da066e22c80b9018a64fbb.tar.gz | |
Misc minor issues caught by cppcheck
All trivial (unused variables, duplicate `break`s, etc).
Change-Id: Idbfffae4f6c0b0119a90ae5849de2ed7a1180c9b
Reviewed-on: https://code.wireshark.org/review/11886
Petri-Dish: Evan Huus <eapache@gmail.com>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'ui/win32')
| -rw-r--r-- | ui/win32/print_win32.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ui/win32/print_win32.c b/ui/win32/print_win32.c index d58bedcec3..cfa211cb8e 100644 --- a/ui/win32/print_win32.c +++ b/ui/win32/print_win32.c @@ -182,7 +182,6 @@ static void print_file(const char *file_name, HDC hdc, int width, int height) #define Y_OFFSET 5 FILE* fh1; - size_t results; int cnt=0, y_pos = Y_OFFSET, y_cnt = 0; char buf[MAX_BUF_SIZE]; char ch; @@ -204,7 +203,7 @@ static void print_file(const char *file_name, HDC hdc, int width, int height) return; } - while ((results = fread(&ch, 1, 1, fh1)) != 0) { + while (fread(&ch, 1, 1, fh1) != 0) { /* end of page (form feed)? */ if (ch == 0x0c) { /* send buffer */ |