Hi,

On 11 July 2015 at 01:58, Bryce Harrington <br...@osg.samsung.com> wrote:
> On Mon, Jun 22, 2015 at 05:25:09PM +0100, Daniel Stone wrote:
>> +     ret = drmModeConnectorSetProperty(c->drm.fd, output->connector_id,
>> +                                       output->dpms_prop->prop_id, level);
>> +     if (ret) {
>
> Just a nitpick, but we don't actually use ret for anything other than
> this test.  Maybe save the variable and just do "if
> (drmModeConnectorSetProperty(..." ?

I'd actually prefer not to, just for readability/consistency's sake.
Thanks though. :)

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to