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

[Derek Foreman]

On 27/03/15 07:38 AM, Pekka Paalanen wrote:
> 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, will do that instead.