Re: [Xen-devel] [PATCH] libxl: const correctness for libxl__xs_path_cleanup

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Thu, 2012-08-02 at 15:17 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[PATCH] libxl: const correctness for 
> libxl__xs_path_cleanup"):
> > libxl: const correctness for libxl__xs_path_cleanup
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > 
> > diff -r e638a0aeb985 -r 075da4778b0a tools/libxl/libxl_device.c
> > --- a/tools/libxl/libxl_device.c    Thu Aug 02 09:59:33 2012 +0100
> > +++ b/tools/libxl/libxl_device.c    Thu Aug 02 10:22:28 2012 +0100
> > @@ -523,7 +523,7 @@ DEFINE_DEVICES_ADD(nic)
> >  int libxl__device_destroy(libxl__gc *gc, libxl__device *dev)
> >  {
> >      char *be_path = libxl__device_backend_path(gc, dev);
> > -    char *fe_path = libxl__device_frontend_path(gc, dev);
> > +    const char *fe_path = libxl__device_frontend_path(gc, dev);
> 
> I don't understand why this is needed (or a good thing) for one but
> not the other.

be_path becomes const in the next patch, after the non-const user is
removed.




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel