Cleanup some cleanup / error paths

[libvirt PATCH 00/10] Cleanup some cleanup / error paths

Tim Wiederhake posted 10 patches 2 years, 5 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20211108121740.98621-1-twiederh@redhat.com

src/admin/admin_server.c       |  4 ++--
src/ch/ch_domain.c             | 34 ++++++++++++++++------------------
src/conf/domain_capabilities.c | 10 +++-------
src/conf/network_event.c       |  9 +++------
src/conf/virsavecookie.c       |  9 ++-------
src/util/virbuffer.c           | 10 ++++------
src/util/virscsivhost.c        |  7 +------
tests/domaincapstest.c         | 18 +++++-------------
tests/testutilslxc.c           | 17 +++++------------
tests/vshtabletest.c           | 12 +++++-------
10 files changed, 46 insertions(+), 84 deletions(-)

Expand all Fold all

[libvirt PATCH 00/10] Cleanup some cleanup / error paths

Posted by Tim Wiederhake 2 years, 5 months ago


Tim Wiederhake (10):
  adminConnectListServers: Cleanup
  virCHDomainObjBeginJob: Cleanup
  virDomainCapsCPUModelsCopy: Cleanup
  virNetworkEventDispatchDefaultFunc: Cleanup
  virSaveCookieParse: Cleanup
  virBufferAddBuffer: Cleanup
  virSCSIVHostOpenVhostSCSI: Cleanup
  fillXenCaps: Cleanup
  testLXCCapsInit: Cleanup
  testVshTableHeader: Cleanup

 src/admin/admin_server.c       |  4 ++--
 src/ch/ch_domain.c             | 34 ++++++++++++++++------------------
 src/conf/domain_capabilities.c | 10 +++-------
 src/conf/network_event.c       |  9 +++------
 src/conf/virsavecookie.c       |  9 ++-------
 src/util/virbuffer.c           | 10 ++++------
 src/util/virscsivhost.c        |  7 +------
 tests/domaincapstest.c         | 18 +++++-------------
 tests/testutilslxc.c           | 17 +++++------------
 tests/vshtabletest.c           | 12 +++++-------
 10 files changed, 46 insertions(+), 84 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH 00/10] Cleanup some cleanup / error paths

Posted by Michal Prívozník 2 years, 5 months ago

On 11/8/21 1:17 PM, Tim Wiederhake wrote:
> 
> 
> Tim Wiederhake (10):
>   adminConnectListServers: Cleanup
>   virCHDomainObjBeginJob: Cleanup
>   virDomainCapsCPUModelsCopy: Cleanup
>   virNetworkEventDispatchDefaultFunc: Cleanup
>   virSaveCookieParse: Cleanup
>   virBufferAddBuffer: Cleanup
>   virSCSIVHostOpenVhostSCSI: Cleanup
>   fillXenCaps: Cleanup
>   testLXCCapsInit: Cleanup
>   testVshTableHeader: Cleanup
> 
>  src/admin/admin_server.c       |  4 ++--
>  src/ch/ch_domain.c             | 34 ++++++++++++++++------------------
>  src/conf/domain_capabilities.c | 10 +++-------
>  src/conf/network_event.c       |  9 +++------
>  src/conf/virsavecookie.c       |  9 ++-------
>  src/util/virbuffer.c           | 10 ++++------
>  src/util/virscsivhost.c        |  7 +------
>  tests/domaincapstest.c         | 18 +++++-------------
>  tests/testutilslxc.c           | 17 +++++------------
>  tests/vshtabletest.c           | 12 +++++-------
>  10 files changed, 46 insertions(+), 84 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal