Looks obviously correct, go ahead and commit it. > Found by gcc's -Wenum-compare. Is this option new, or newly applicable to C? (The manual I have says "(C++ and Objective-C++ only)".) Is it on by default, or should we be passing it? Thanks, Roland