search

rpodgorny / unionfs-fuse

union filesystem using fuse
Other
300 stars 76 forks source link

use correct boolean operator #77

Closed mmcco closed 6 years ago

mmcco commented 6 years ago

In its current state, this condition doesn't make sense and will always be false. You probably meant to use the || operator.

mmcco commented 6 years ago

I see.

Anyway, I'll leave it to you to use (void)arg if you want to. IMO, that'd be a worthwhile simplification. It makes the code more readable, and compilers will probably start warning about that condition at some point (I'm considering attempting to write the warning logic for Clang in a couple weeks.)