[PATCH:xman 1/2] Fix tautology to the intended check.

Thomas Klausner wiz at NetBSD.org
Fri May 23 08:09:25 PDT 2014


On Fri, May 23, 2014 at 10:10:19AM +0200, walter harms wrote:
> 
> 
> Am 23.05.2014 07:44, schrieb Alan Coopersmith:
> > On 05/21/14 03:17 PM, Thomas Klausner wrote:
> >> From: Jörg Sonnenberger <joerg at NetBSD.org>
> >>
> >> Signed-off-by: Thomas Klausner <wiz at NetBSD.org>
> >> ---
> >>   handler.c | 2 +-
> >>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/handler.c b/handler.c
> >> index e61687e..87a928e 100644
> >> --- a/handler.c
> >> +++ b/handler.c
> >> @@ -320,7 +320,7 @@ SaveFormattedPage(Widget w, XEvent * event, String
> >> * params,
> >>    * If we are not active then take no action.
> >>    */
> >>
> >> -    if (man_globals->tempfile == NULL)
> >> +    if (man_globals->tempfile[0] == '\0')
> >>           return;
> >>
> >>       switch (params[0][0]) {
> >>
> > 
> > Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> > 
> 
> Hi,
> IMHO this is not equal, can you explain that NULL is not possible ?

Look at the definition of man_globals, its type is ManpageGlobals,
and the tempfile member of ManpageGlobals is declared this way:

    char tempfile[80];

Since tempfile is a fixed-size array, it can never be a NULL pointer,
so the comparison as-is doesn't make sense.
 Thomas


More information about the xorg-devel mailing list