Question

while (curr_data[1] != (unsigned int)NULL &&
    ((curr_ptr = (void*)curr_data[1]) || 1))

Two part question.

What will (curr_ptr = (void*)curr_data[1]) evaluate to, logically. TRUE?

Also, I know its rather hack-ish, but is the while statement legal C? I would have to go through great contortions to put the assignment elsewhere in the code, so I’d be really nice if I could leave it there, but if it’s so egregious that it makes everyone’s eyeballs burst into flames, I’ll change it.

Answer 1

To answer your questions:

1. It will evaluate to true if curr_ptr isn’t set to NULL (i.e. curr_data[1] isn’t 0).
2. I believe it’s legal, but there are bigger problems with this line of code.

Anyway, I’m assuming you didn’t write this code, because you’re debating about leaving it in vs. taking it out. So I want you to find out who wrote this line of code and introduce them to a heavy blunt object.

1. (unsigned int)NULL is ridiculous. Why would you do this? This will probably be the same as just writing 0 (not sure if that’s guaranteed by the standard).
2. What kind of data is in curr_data[1] if it’s being cast to a pointer (and pointers are being cast to it)? If it’s supposed to be holding a pointer as an integral type, you should use the type intptr_t or uintptr_t provided in <stdint.h> for that purpose (if you’re compiler doesn’t support C99 ptrdiff_t may be an acceptable substitute).
3. The || 1 at the end seems to be redundant. If curr_ptr = (void*)curr_data[1] would have evaluated to false, we would have caught that in the first condition.

It may be a pain in the ass, but seriously reconsider rewriting this line. It looks like an entry in the IOCCC.