Juan Manuel Guerrero dixit:

> +#ifdef USE_PERSISTENT_COOKIES
>     if (LYCookieFile != 0 && LYCookieSaveFile != 0) {
> +#else
> +    {

Shouldn’t that be an “if (0) {” instead?

> +#endif
>       /* set few safe flags: */
> #ifdef USE_PERSISTENT_COOKIES
>       BOOLEAN persistent_cookies_flag = persistent_cookies;
>       char *LYCookieFile_flag = NULL;
>       char *LYCookieSaveFile_flag = NULL;

Also, consider something like this:

#ifdef USE_PERSISTENT_COOKIES
   if (LYCookieFile != 0 && LYCookieSaveFile != 0)
#else
   if (0)
#endif
     {
…
   }

With the { a bit indented from each of the ifs, but less
than the block inside, to make editors’ jump-to-matching-
brace functions not fuck up.

bye,
//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"

_______________________________________________
Lynx-dev mailing list
Lynx-dev@nongnu.org
https://lists.nongnu.org/mailman/listinfo/lynx-dev

Reply via email to