[sheepdog] [PATCH v3 2/2] sheep: implement atomic create and write mechanism for config
Liu Yuan
namei.unix at gmail.com
Tue Apr 16 10:42:46 CEST 2013
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
More information about the sheepdog
mailing list