[PATCH wayland v2] util: Clarify documentation of wl_dispatcher_func_t

Daniel Stone daniel at fooishbar.org
Mon Nov 21 21:17:14 UTC 2016


Hi,

On 21 November 2016 at 20:59, Yong Bakos <junk at humanoriented.com> wrote:
> On Nov 21, 2016, at 9:14 AM, Daniel Stone <daniel at fooishbar.org> wrote:
>> I think removing the final word 'the dispatcher-specific
>> implementation data' is a loss of precision/accuracy. If you don't
>> mind reinstating that final word, I'll merge with my R-b.
>
> I have no problem with this. The reason why I changed it from "implementation data"
> to just "implementation" is because wl_closure_dispatch[1] passes an implementation
> as the first argument. I assumed this use was the specific intent, making the
> description vague; but perhaps this is just one intent, and the description should
> be more general ("implementation data").
>
> I'm fine either way (you know waaaay better than I do).

OK, fair enough! Pushed in its original form, thanks:

To ssh://git.freedesktop.org/git/wayland/wayland
   a26ed09..0242007  master -> master

Cheers,
Daniel


More information about the wayland-devel mailing list