<p dir="ltr"><br>
On Mar 18, 2014 2:09 PM, "Bryce W. Harrington" <<a href="mailto:b.harrington@samsung.com">b.harrington@samsung.com</a>> wrote:<br>
><br>
> On Mon, Mar 17, 2014 at 11:34:50AM +0800, Quanxian Wang wrote:<br>
> > This event contains name of output. It may be sent after<br>
> > binding the output object. It is intended that client could<br>
> > input a character output name as a parameter or for log output.<br>
> ><br>
> > Signed-off-by: Quanxian Wang <<a href="mailto:quanxian.wang@intel.com">quanxian.wang@intel.com</a>><br>
> > ---<br>
> >  protocol/wayland.xml | 11 +++++++++++<br>
> >  1 file changed, 11 insertions(+)<br>
> ><br>
> > diff --git a/protocol/wayland.xml b/protocol/wayland.xml<br>
> > index d47ee62..0332e1a 100644<br>
> > --- a/protocol/wayland.xml<br>
> > +++ b/protocol/wayland.xml<br>
> > @@ -1756,6 +1756,17 @@<br>
> >        </description><br>
> >        <arg name="factor" type="int" summary="scaling factor of output"/><br>
> >      </event><br>
> > +<br>
> > +    <event name="name" since="3"></p>
<p dir="ltr">You also need to bump the version field on the wl_output interface</p>
<p dir="ltr">> > +      <description summary="output name properties"><br>
> > +     This event contains name of output. It may be sent after<br>
> > +     binding the output object. It will provide useful name<br>
> > +     for client when they generate the log for user.<br>
> > +     Also it is intended that client could input a character<br>
> > +     output name as a parameter or log.<br>
><br>
> Fixing up the grammar a bit:<br>
><br>
> """<br>
> This event contains the name of the output.  It may be sent after<br>
> binding the output object.  It will provide a useful name for the<br>
> client when it generates the log for the user.  Also, it is intended<br>
> that the client could input a character output name as a parameter or<br>
> log.<br>
> """<br>
><br>
> If I understand the purpose of this patchset properly, this might be<br>
> clearer phrasing:<br>
><br>
> """<br>
> This event allows assigning a human-readable alias to an output; it may<br>
> be sent after binding the output object.  It is intended that the client<br>
> can use this output name as a parameter or display it in logs.<br>
> """<br>
><br>
> I glanced at the other patches and they look technically correct, but I<br>
> think what might be missing here is a stronger justification about why<br>
> this is needed?</p>
<p dir="ltr">While I kind of like the idea of a human-readable name, I agree with Bryce that some justification is needed.</p>
<p dir="ltr">Thanks,<br>
Jason Ekstrand</p>
<p dir="ltr">><br>
> > +      </description><br>
> > +      <arg name="name" type="string" summary="name of output"/><br>
> > +    </event><br>
> >    </interface><br>
> ><br>
> >    <interface name="wl_region" version="1"><br>
> > --<br>
> > 1.8.1.2<br>
> ><br>
> > _______________________________________________<br>
> > wayland-devel mailing list<br>
> > <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a></p>