FreeTDS Development Group

["James K. Lowden" <jklowden AT schemamania.org>]

http://www.fiendish.demon.co.uk/c/casting.html
http://www.vmunix.com/~gabor/c/draft.html#6.2.2.3

I came across this line in bcp.c (reformatted):

if ((curptr->nextptr = 
        (struct fflist *) malloc(sizeof(struct fflist))) 
                == (struct fflist *) NULL)

ANSI C says:

1.  It's never necessary to cast to compare to a null pointer.  
2.  It's never necessary to cast void* for assignment.  

That tells me this could as well read:

if ((curptr->nextptr = 
        malloc(sizeof(struct fflist))) == NULL)

which I can actually read.  Is there any reason not to simplify this?  Is
anyone really using a malloc() that returns char* in 2004?  (The casting
of NULL is just plain egregious, not merely old fashioned.)

--jkl