On Mon, Apr 13, 2015 at 2:43 PM, Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> wrote: > Paul Tan <pyokagan@xxxxxxxxx> 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 <Matthieu.Moy@xxxxxxx> and Reviewed-by: Stefan Beller <sbeller@xxxxxxxxxx> > > -- > 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 majordomo@xxxxxxxxxxxxxxx > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html