lynx-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Lynx-dev] Re: Bug#509321: lynx-cur: Visiting http://reddit.com/ cau


From: Paul Gilmartin
Subject: Re: [Lynx-dev] Re: Bug#509321: lynx-cur: Visiting http://reddit.com/ causes segfault.
Date: Wed, 24 Dec 2008 08:55:15 -0700


On Dec 23, 2008, at 11:35, Thorsten Glaser wrote:

Thomas Dickey dixit:

+    if (ct == 0) {
+       method = cftNone;
+    } else if (!strncasecomp(ct, "application/gzip", 16) ||

Is it a string or a number? Please use NULL, not 0.

If it's a string or notionally logical, I'm quite comfortable
with "if ( ! ct ) ..." (or "if ( ! *ct ) ...", depending).
If it's notionally numeric, I'm more comfortable with "if (ct == 0) ..."

-- gil





reply via email to

[Prev in Thread] Current Thread [Next in Thread]