[Evolution-hackers] Logic flaw in header_decode_lwsp()

[Jules Colding <colding omesc com>]

Hi,

The following while statement does not make sense:

############# snip ###########
static void
header_decode_lwsp(const char **in)
{
	const char *inptr = *in;
	char c;

	d2(printf("is ws: '%s'\n", *in));

	while (camel_mime_is_lwsp(*inptr) || (*inptr =='(' && *inptr != '\0')) {
        .
        .
        .

############# snip ###########

If *inptr is equal to '(' then it is per definition not equal to '\0'. 

OK, "does not makes sense" might to harsh, but it definitely does not
seem to done this way on purpose. 


-- 
  jules