[Lynx-dev] maybe better HTContentTypeToCompressType patch?

[Thorsten Glaser]

HTEncodingToCompressType has this redundant, should maybe fixed too:
• the { } are not needed
• assigning to result is redundant, it’s already initialised to cftNone
• we can shortcut the return

Index: HTFile.c
===================================================================
RCS file: /cvs/src/gnu/usr.bin/lynx/WWW/Library/Implementation/HTFile.c,v
retrieving revision 1.1.109.6
diff -u -p -r1.1.109.6 HTFile.c
--- HTFile.c    16 Dec 2008 23:13:07 -0000      1.1.109.6
+++ HTFile.c    26 Dec 2008 13:43:09 -0000
@@ -1336,6 +1336,8 @@ CompressFileType HTContentTypeToCompress
 {
     CompressFileType method = cftNone;
 
+    if (ct == NULL)
+       return cftNone;
     if (!strncasecomp(ct, "application/gzip", 16) ||
        !strncasecomp(ct, "application/x-gzip", 18)) {
        method = cftGzip;


//mirabilos
-- 
Sometimes they [people] care too much: pretty printers [and syntax highligh-
ting, d.A.] mechanically produce pretty output that accentuates irrelevant
detail in the program, which is as sensible as putting all the prepositions
in English text in bold font.   -- Rob Pike in "Notes on Programming in C"