Re: [PATCH v6 42/42] proto: Describe Quic image format from dissector

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 13, 2015 at 02:12:21PM +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  spice.proto | 25 ++++++++++++++++++++++++-
>  1 file changed, 24 insertions(+), 1 deletion(-)
> 
> diff --git a/spice.proto b/spice.proto
> index c7944ac..6a51645 100644
> --- a/spice.proto
> +++ b/spice.proto
> @@ -687,6 +687,29 @@ struct Surface {
>      uint32 surface_id;
>  } @ws_txt("Surface ID: %u", surface_id);
>  
> +enum32 quic_image_type {
> +    INVALID,
> +    GRAY,
> +    RGB16,
> +    RGB24,
> +    RGB32,
> +    RGBA
> +} @ws("QUIC image type", quic_type) @prefix(WSQUIC_IMAGE_TYPE_) @ifdef(DISSECTOR);
> +
> +struct ImageQuic {
> +    uint32 magic @ws_desc("QUIC magic (QUIC)");
> +    uint16 major @ws("QUIC major version", quic_major_version);
> +    uint16 minor @ws("QUIC minor version", quic_minor_version);
> +    quic_image_type type;
> +    uint32 width @ws("Width", quic_width);
> +    uint32 height @ws("Height", image_height);
> +    uint8 data[] @end @ws_txt("QUIC compressed image data (%u bytes)", data.nelements);
> +};

Would it make sense to annotate the structs with @ifdef too ?

Christophe

Attachment: pgpdrrwU52dP1.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]