Re: Austin English : shell32/tests: Don't check return values inside of if(0) ( LLVM/Clang).

[Charles Davis]

Austin,

Static analyzer checks for unreachable assignments (like the ones that
were removed in this patch) were turned off in Clang trunk recently (as
of r125415). Looks like Clang was wrong ;).

Chip

Austin English : shell32/tests: Don't check return values inside of if(0) ( LLVM/Clang).

[Michael Stefaniuc]

Charles,

Charles Davis wrote:
> Austin,
>
> Static analyzer checks for unreachable assignments (like the ones that
> were removed in this patch) were turned off in Clang trunk recently (as
> of r125415). Looks like Clang was wrong ;).

the ones that Austin removed are still pointless. The code is there to
document the crashing function call. And a crashing function call has no
return value ;)

bye
    michael