[PATCH weston v2 3/6] test: Add clock_settime()

[Pekka Paalanen]

On Mon, 22 Dec 2014 16:09:58 -0600
Derek Foreman <derekf at osg.samsung.com> wrote:

> Add a way to set the presentation clock to an arbitrary value, but only
> when in test mode.
> 
> Signed-off-by: Derek Foreman <derekf at osg.samsung.com>
> ---
>  protocol/wayland-test.xml |  9 +++++++++
>  src/compositor.c          | 23 +++++++++++++++++++++++
>  src/compositor.h          |  4 ++++
>  tests/weston-test.c       | 13 +++++++++++++
>  4 files changed, 49 insertions(+)
> 
> diff --git a/protocol/wayland-test.xml b/protocol/wayland-test.xml
> index 292abe9..70016ce 100644
> --- a/protocol/wayland-test.xml
> +++ b/protocol/wayland-test.xml
> @@ -68,5 +68,14 @@
>          Once test mode is enabled it remains active for the life of the
>          compositor.
>        </description>
> +    <request name="clock_settime">
> +      <description summary="set the presentation clock time">
> +        Sets the presentation clock time.
> +
> +        Requires compositor test mode to be active.
> +      </description>
> +      <arg name="sec" type="string" summary="whole part in seconds"/>
> +      <arg name="nsec" type="string" summary="fractional part in seconds"/>

Why strings? Why not uints, e.g.
      <arg name="tv_sec_hi" type="uint"
           summary="high 32 bits of the seconds part of the presentation timestamp"/>
      <arg name="tv_sec_lo" type="uint"
           summary="low 32 bits of the seconds part of the presentation timestamp"/>
      <arg name="tv_nsec" type="uint"
           summary="nanoseconds part of the presentation timestamp"/>
like the Presentation extension uses?


Thanks,
pq