[v2] Fix up some issues from x and y resolution patches

[libvirt] [PATCH v2 0/5] Fix up some issues from x and y resolution patches

Jonathon Jongsma posted 5 patches 5 years, 1 month ago

Diff against v1 v3
Download series mbox

Test syntax-check passed

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

There is a newer version of this series

docs/formatdomain.html.in |  12 +++--
src/conf/domain_conf.c    | 108 ++++++++++++++++++++++----------------
2 files changed, 71 insertions(+), 49 deletions(-)

Expand all Fold all

[libvirt] [PATCH v2 0/5] Fix up some issues from x and y resolution patches

Posted by Jonathon Jongsma 5 years, 1 month ago

This is a follow-up series responding to some comments from Jan Tomko. Most
importantly, the fact that the errors are not propagated up to the caller,
they're only logged. To fix this, the function signatures were changed to
return a error status.

The patch series was also re-ordered slightly to improve readability (I hope).

Jonathon Jongsma (5):
  qemu: fix documentation for video resolution
  conf: remove unnecessary NULL checks
  conf: use glib allocation when parsing video props
  conf: report errors when parsing video resolution
  conf: report errors when parsing video acceleration

 docs/formatdomain.html.in |  12 +++--
 src/conf/domain_conf.c    | 108 ++++++++++++++++++++++----------------
 2 files changed, 71 insertions(+), 49 deletions(-)

-- 
2.21.0

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