Daniel Shahaf wrote on Mon, Oct 03, 2011 at 01:13:52 +0200: > + if (root_noderev->predecessor_count != -1 > + && root_noderev->predecessor_count != rev);
Nothing to see here; stray semicolon. I disable "Will never be executed" warnings so my compiler didn't warn me about it. Building on another box triggered the warning.