On Mon, Apr 13, 2015 at 2:43 PM, Matthieu Moy <[email protected]> wrote: > Paul Tan <[email protected]> writes: > >> As such, implement a simpler function xdg_config_home() for constructing >> the XDG base dir spec configuration file path. This function, together >> with expand_user_path(), can replace all uses of home_config_paths(). > > Indeed. The code looks much better after your patch series than before. > > I agree with Dscho's remark that "fn" sounds like "function" more than > "filename". Perhaps just "name" would be better.
I'd go with fname but that's just bikeshedding now. > > Anyway, the series is > > Reviewed-by: Matthieu Moy <[email protected]> and Reviewed-by: Stefan Beller <[email protected]> > > -- > Matthieu Moy > http://www-verimag.imag.fr/~moy/ > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
