[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v2 7/8] qapi: golang: Add CommandResult type to Go
From: |
Victor Toso |
Subject: |
Re: [RFC PATCH v2 7/8] qapi: golang: Add CommandResult type to Go |
Date: |
Wed, 17 Aug 2022 17:00:26 +0200 |
Hi,
On Tue, Jul 05, 2022 at 05:49:22PM +0100, Daniel P. Berrangé wrote:
> On Tue, Jul 05, 2022 at 08:46:21AM -0700, Andrea Bolognani wrote:
> > On Fri, Jun 17, 2022 at 02:19:31PM +0200, Victor Toso wrote:
> > > +type EmptyCommandReturn struct {
> > > + CommandId string `json:"id,omitempty"`
> > > + Error *QapiError `json:"error,omitempty"`
> > > + Name string `json:"-"`
> > > +}
> >
> > Do we need a specific type for this? Can't we just generate an
> > appropriately-named type for each of the commands that don't return
> > any data? It's not like we would have to write that code manually :)
>
> Yes, I think having an explicit named return struct even for commands
> not /currently/ returning data is good, and anticipates future changes
> that might add extra return data fields to the QAPI schema.
Sure.
Cheers,
Victor
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [RFC PATCH v2 7/8] qapi: golang: Add CommandResult type to Go,
Victor Toso <=