At Tue, 16 Apr 2013 16:42:46 +0800, Liu Yuan wrote: > > 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.
Ah, I see. Let's remove the variable. I'll send v4 later. Thanks, Hitoshi -- sheepdog mailing list sheepdog@lists.wpkg.org http://lists.wpkg.org/mailman/listinfo/sheepdog