On Monday, March 11, 2019 9:53 PM, Harish Krupo <[email protected]> 
wrote:
> Thanks for the comments. Please find my comments inline.

Thank you for working on this!

[…]

> >> +      An extension interface to the wl_surface object to set the HDR 
> >> metadata
> >> +      associated with the surface.
> >> +    </description>
> >> +
> >> +    <enum name="EOTF">
> >
> > Style nit: this can be lowercase.
> >
>
> Enum names lowercase?

Yeah, that's the convention we've been using so far.

[…]

> Thanks again for reviewing. As this is just a provisional protocol, we
> didn't give much heed to the exact details. I agree that it shouldn't be
> the case. I will update this protocol with more details and send in v2.

I see. No rush then, I'll wait for the next version. :)

I have similar comments for the second part of this patchset, but I'm going to
hold off until v2 is sent.
_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to