operator precedence means ok, but I'd prefer braces around 2nd expression (... != 0)