[PATCH 3/3] virfile: Use g_canonicalize_file() to simplify virFileAbsPath()

Luke Yue lukedyue at gmail.com
Fri Jun 4 11:38:42 UTC 2021


On Fri, 2021-06-04 at 12:28 +0200, Martin Kletzander wrote:
> On Mon, May 31, 2021 at 09:48:24AM +0800, Luke Yue wrote:
> > Though the comment says that the function may return -1 on error,
> > but it
> > seems that now it will never return -1 now. So just use
> > g_canonicalize_file()
> > to simplify the implementation.
> > 
> 
> Yeah, that is a leftover from before we started using glib and
> abort()'ing on OOM.  It would be nice if that leftover got cleaned up
> as
> well.  However, looking at it, we can remove the function altogether
> and
> just use the glib counterpart.  That'd be even more of a clean up ;)

Thanks for the review!

I will try to replace all the funciton calls with g_canonicalize_file()
and remove the function in a new patch.

Luke






More information about the libvir-list mailing list