> On Sat, Apr 2, 2016 at 2:48 PM, Yong Bakos <j...@humanoriented.com> wrote: > From: Yong Bakos <yba...@humanoriented.com> > > This v2 patch incorporates some recent feedback per v1. Two things to note: > > 1) The 'name' parameter in wl_registry events has been changed to "numeric > name" > to indicate that, "yes, this is numeric but it serves as a name." > This is a "best option" measure to put /something/ here to squelch the > warning, which is the main goal of this patch. The semantics of the name > 'name' is another concern that I feel does need addressed, but not here. > > On Apr 4, 2016, at 2:23 PM, Bill Spitzak <spit...@gmail.com> wrote: > How about "server's id"? > > I don't think the word "name" should be in the description at all, as it > strongly implies that the data is a string.
I agree that this needs addressed, "but not here." For example, pq has asserted that the name argument is not an id. While I don't entirely agree, I'm a noob and I feel more thought needs to go into how this argument is used, what values are assigned to it, how the name property of the corresponding struct is used elsewhere, what to rename it to, and whether or not it's worth the repercussions. The description will be the easy part. I am only now digging into this, so if you already know the answers, please educate me! I'd like to get this patch merged to suppress the `make check` warnings, and then submit an intelligent patch that addresses this 'name' parameter everywhere it occurs. yong _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel
