It's not wrong but when I see the code I want to refine it since it looks
uncomfortable.
 680     if (Config.Announce.period > 0) {
 681         Config.onoff.announce = 1;
 682     } else if (Config.Announce.period < 1) {
 683         Config.Announce.period = 86400 * 365;   /* one year */
 684         Config.onoff.announce = 0;
 685     }
Announce.period is with type "time_t" so it's an integer. The "else-if" is
"logically" superfluous because as long as the value <=0, it must be <1.
Best,
Tianyin
-- Tianyin XU, http://cseweb.ucsd.edu/~tixu/
This archive was generated by hypermail 2.2.0 : Fri Feb 08 2013 - 12:00:07 MST