TS>  JdBP> if (!terminating) {
 TS>  JdBP> } else {

 TS> I don't think it is good style to handle both cases of 
 TS> a boolean decision and begin with the negated part. 
 TS> Probably the compiler can optimize it, but I wouldn't 
 TS> trust this :)

 TS> if (terminating) {}
 TS> else {}

 TS> is clearer, although your style fits the causality better :-)

Generally people try to put the error handling (and other less-frequently
run) code later.  "Terminating" is likely an "error"
(or at least not the normal path of execution), and probably should be
handled later.  OTOH, I prefer my variables to be positive as well - rather
than "terminating," have "still_going" or
"alive" or even (if all is lost), "not_terminating". 
:-)  You win both ways - you don't continually negate the variable, and you
still get to put the most-executed code first where we can read it easier. 
:-)


--- Maximus/2 3.01