Re: [PATCH v14 0/5] MR3103: server: Create and use a desktop shared mapping for GetCursorPos.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about server/protocol.def:
>  lparam_t info;
>
> } cursor_pos_t;
>
> +struct shared_cursor
> +{
> - int                  x;                /* cursor position */
> - int                  y;
> - unsigned int         last_change;      /* time of last position change */
>
> +};
> -
>
> +struct desktop_shared_memory
> +{
> - unsigned int         seq;              /* sequence number - server updating if (seq & 1) != 0 */
> - struct shared_cursor cursor;           /* global cursor information */
>
> +};
> +typedef volatile struct desktop_shared_memory desktop_shm_t;

How about merging the declaration?

```suggestion:-5+0
typedef volatile struct desktop_shared_memory
{
    unsigned int         seq;              /* sequence number - server updating if (seq & 1) != 0 */
    struct shared_cursor cursor;           /* global cursor information */
} desktop_shm_t;
```

Note that this doesn't make `struct desktop_shared_memory` itself volatile.