diff options
author | Simon Tatham <anakin@pobox.com> | 2020-01-29 09:35:53 +0300 |
---|---|---|
committer | Simon Tatham <anakin@pobox.com> | 2020-01-29 09:44:18 +0300 |
commit | 76430f8237c1b0027c86e51cd675270abe155ee0 (patch) | |
tree | be457abd42e939ad4545049546e7acba233d70c0 /logging.c | |
parent | 8d747d802900aed50f9ff7e0bef0f560a0ad72ef (diff) |
Assorted benign warning fixes.
These were just too footling for even me to bother splitting up into
multiple commits:
- a couple of int -> size_t changes left out of the big-bang commit
0cda34c6f
- a few 'const' added to pointer-type casts that are only going to be
read from (leaving out the const provokes a warning if the pointer
was const _before_ the cast)
- a couple of 'return' statements trying to pass the void return of
one function through to another.
- another missing (void) in a declaration in putty.h (but this one
didn't cause any knock-on confusion).
- a few tweaks to macros, to arrange that they eat a semicolon after
the macro call (extra do ... while (0) wrappers, mostly, and one
case where I had to do it another way because the macro included a
variable declaration intended to remain in scope)
- reworked key_type_to_str to stop putting an unreachable 'break'
statement after every 'return'
- removed yet another type-check of a function loaded from a Windows
system DLL
- and finally, a totally spurious semicolon right after an open brace
in mainchan.c.
Diffstat (limited to 'logging.c')
-rw-r--r-- | logging.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -364,7 +364,7 @@ void log_packet(LogContext *ctx, int direction, int type, c = 'X'; sprintf(smalldata, "XX"); } else { /* PKTLOG_EMIT */ - c = ((unsigned char *)data)[p]; + c = ((const unsigned char *)data)[p]; sprintf(smalldata, "%02x", c); } dumpdata[10+2+3*(p%16)] = smalldata[0]; |