+> Code cleanups:
+ <P
+></P
+><UL
+><LI
+><P
+> Remove the next member from the client_state struct. Only the main
+ thread needs access to all client states so give it its own struct.
+ </P
+></LI
+><LI
+><P
+> Garbage-collect request_contains_null_bytes().
+ </P
+></LI
+><LI
+><P
+> Ditch redundant code in unload_configfile().
+ </P
+></LI
+><LI
+><P
+> Ditch LogGetURLUnderCursor() which doesn't seem to be used anywhere.
+ </P
+></LI
+><LI
+><P
+> In write_socket(), remove the write-only variable write_len in
+ an ifdef __OS2__ block. Spotted by cppcheck.
+ </P
+></LI
+><LI
+><P
+> In connect_to(), don't declare the variable 'flags' on OS/2 where
+ it isn't used. Spotted by cppcheck.
+ </P
+></LI
+><LI
+><P
+> Limit the scope of various variables. Spotted by cppcheck.
+ </P
+></LI
+><LI
+><P
+> In add_to_iob(), turn an interestingly looking for loop into a
+ boring while loop.
+ </P
+></LI
+><LI
+><P
+> Code cleanup in preparation for external filters.
+ </P
+></LI
+><LI
+><P
+> In listen_loop(), mention the socket on which we accepted the
+ connection, not just the source IP address.
+ </P
+></LI
+><LI
+><P
+> In write_socket(), also log the socket we're writing to.
+ </P
+></LI
+><LI
+><P
+> In log_error(), assert that escaped characters get logged
+ completely or not at all.
+ </P
+></LI
+><LI
+><P
+> In log_error(), assert that ival and sval have reasonable values.
+ There's no reason not to abort() if they don't.
+ </P
+></LI
+><LI
+><P
+> Remove an incorrect cgi_error_unknown() call in a
+ cannnot-happen-situation in send_crunch_response().
+ </P
+></LI
+><LI
+><P
+> Clean up white-space in http_response definition and
+ move the crunch_reason to the beginning.
+ </P
+></LI
+><LI
+><P
+> Turn http_response.reason into an enum and rename it
+ to http_response.crunch_reason.
+ </P
+></LI
+><LI
+><P
+> Silence a 'gcc (Debian 4.3.2-1.1) 4.3.2' warning on i686 GNU/Linux.
+ </P
+></LI
+></UL
+>