On 04/16/2013 04:28 PM, Hitoshi Mitake wrote: > + snprintf(tmp_config_path, len, "%s" CONFIG_PATH ".tmp", base_path);
Why not remove this tmp_config_path too? We should always use global variable as less as much. Only write_config() and check_tmp_config() need it, and both of them can deduce tmp_config_path from config_path, so I think it is better to get tmp path on the fly instead of a global variable. Thanks, Yuan -- sheepdog mailing list [email protected] http://lists.wpkg.org/mailman/listinfo/sheepdog
