On 12/18/19 3:17 PM, Fabiano Fidêncio wrote: > Let's use the defines provided by GLib, instead of keeping our own. > > Fabiano Fidêncio (2): > util: Use G_DIR_SEPARATOR instead of VIR_FILE_DIR_SEPARATOR > util: Remove VIR_FILE_*_SEPARATOR* > > src/util/virfile.c | 2 +- > src/util/virfile.h | 19 ------------------- > 2 files changed, 1 insertion(+), 20 deletions(-) > I'm guessing this series is based on your previous virFile removals. An independent series should be based on master, otherwise 'git am' gets confused about missing ancestors which makes it harder to test+review. I applied manually though, so: Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> Possible follow up: virFileRemoveLastComponent only has one user, and it may be pretty close to g_path_get_dirname(), so that can possibly be dropped. Might want to check the glib implementation though - Cole -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
