Re: [libvirt] [PATCH]: Fixed minor bugs in display and added OOM checks.

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

 



> > Lastly added OOM checks in few places which were missed earlier.
>
>   Okay applied thanks !
> I would concur with Chris, I think the current practice in most
> of libvirt code when there is a complex routine is to handle
> OOM with a goto to an error label.

Thanks Chris and Daniel for the review, i will change the function here as 
well as others where there are complex routines and make it like the other 
drivers in libvirt, will soon post a patch for same.

> This allows to limit the level
> of cascading. This is stylistic, but affects readability, and
> hence maintainability :-)

stylistic, ha ha.

Thanks and Regards,
Pritesh

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[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]