[libvirt] [PATCH RESEND] Added support for portable-rpcgen from portablexdr library

Martin Kletzander mkletzan at redhat.com
Tue Jul 7 13:26:07 UTC 2015


On Tue, Jul 07, 2015 at 03:04:51PM +0300, Pavel Fedin wrote:
> Hello!
>
>> I don't think we should use enums at all in the protocol
>> specification.  It should be unsigned int and the change should be
>> made separately.  Is this hunk here because portable-rpcgen has
>> problems with enum being declared there?
>
> Yes, its parser fails.
> I have checked throughout the code, there are some other enums in .x files too, but when they are
>put in structs, they are specified without 'enum' keyword and everything is fine. For example,
>virnetprotocol.x:
>--- cut ---
>struct virNetMessageHeader {
>    unsigned prog;              /* Unique ID for the program */
>    unsigned vers;              /* Program version number */
>    int proc;                   /* Unique ID for the procedure within the program */
>    virNetMessageType type;     /* Type of message */
>    unsigned serial;            /* Serial number of message. */
>    virNetMessageStatus status;
>};
>--- cut ---
> Here virNetMessageType and virNetMessageStatus are enums. Actually i borrowed fix method from here.
>

I see.  I'm still a little bit hesitant about that, but that's
pre-existing to your patch, so that shouldn't keep us away from that.
Could you split that hunk into another patch and also fix it in
lxc_monitor_protocol-structs so that make syntax-check doesn't bother
us?

Thanks,
Martin

>Kind regards,
>Pavel Fedin
>Expert Engineer
>Samsung Electronics Research center Russia
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150707/e17bf97e/attachment-0001.sig>


More information about the libvir-list mailing list