On 10/21/20 9:15 AM, Markus Armbruster wrote:
Member 'address' is union GuestDeviceAddress with a single branch
GuestDeviceAddressPCI, containing PCI vendor ID and device ID.  This
is not a PCI address.  Type GuestPCIAddress is.  Messed up in recent
commit 2e4211cee4 "qga: add command guest-get-devices for reporting
VirtIO devices".

Out of curiosity, how did you notice?

Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>


Rename type GuestDeviceAddressPCI to GuestDeviceIdPCI, type
GuestDeviceAddress to GuestDeviceId, and member 'address' to 'id'.

Document the member properly while there.

Signed-off-by: Markus Armbruster <arm...@redhat.com>
---
  qga/qapi-schema.json | 17 +++++++++--------
  qga/commands-win32.c | 16 ++++++++--------
  2 files changed, 17 insertions(+), 16 deletions(-)


Reply via email to