Re: [PATCH v2 32/32] util: Introduce VIR_DEFINE_AUTOPTR_FUNC for virStorageSource

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Feb 11, 2019 at 08:33:32AM -0500, John Ferlan wrote:
>
>
> On 2/11/19 7:44 AM, Erik Skultety wrote:
> > On Fri, Feb 08, 2019 at 01:37:26PM -0500, John Ferlan wrote:
> >> Let's make use of the auto __cleanup capabilities cleaning up any
> >> now unnecessary goto paths.
> >>
> >> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
> >> ---
> > ...
> >>              target->backingStore->type = VIR_STORAGE_TYPE_NETWORK;
> >>              target->backingStore->path = meta->backingStoreRaw;
> >> @@ -3430,8 +3428,6 @@ storageBackendProbeTarget(virStorageSourcePtr target,
> >>      target->format = meta->format;
> >>
> >>      /* Default to success below this point */
> >> -    ret = 0;
> >> -
> >
> > Not sure how much sense the comment makes without ^this hunk, I think we can
> > drop it too
>
> 50/50 coin flip, I can remove
>
> >
> > MinGW is still stuborn about inlining in certain VIR_AUTO cases, I'm wondering
> > why GCC and Clang are okay with that and whether we should drop -Winline or go
> > without the cases that MinGW is sad about, I tried to lookup something related,
> > but it doesn't seem to be a recent bug in MinGW.
> >
> > I'm okay with the changes, but I don't want to have a failing build for the
> > next X weeks, so I'd say drop the violators:
>
> I don't know which violators exist as I don't build MinGW nor do I have
> whatever nifty environment generates that type of build as my normal
> process.

You can either set up your own environment with Andrea's lcitool or you can
make use of github's travis integration:
https://travis-ci.org/eskultety/libvirt/jobs/491591878

Erik


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux