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. Anyway, the series is Reviewed-by: Matthieu Moy <Matthieu.Moy@xxxxxxx> -- 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