On Mon, Sep 12, 2016 at 08:38:32AM -0400, Frediano Ziglio wrote:
> Don't we have an enum for this return instead of using int ?
>
> enum {
> AGENT_MSG_FILTER_OK,
> AGENT_MSG_FILTER_DISCARD,
> AGENT_MSG_FILTER_PROTO_ERROR,
> AGENT_MSG_FILTER_MONITORS_CONFIG,
> AGENT_MSG_FILTER_END
> };
>
> mumble... weird enum. Beside AGENT_MSG_FILTER_END which is
> not used there is OK but also a AGENT_MSG_FILTER_MONITORS_CONFIG
> which I suppose it's OK too... but this is not related to this
> patch.
> Fwiw, iirc AGENT_MSG_FILTER_MONITORS_CONFIG is used to switch between having resolution changes managed by the guest agent (old-style), or directly by QXL (new-style). Odd place to have that, but it's how this was done. Christophe
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/spice-devel
