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/
cache_cf.ifelse.patch
Description: Binary data
