On 2016-08-08 01:55:18, Tomasz Figa wrote:
>    On Fri, Aug 5, 2016 at 2:37 AM, Jordan Justen <jordan.l.jus...@intel.com>
>    wrote:
> 
>      On 2016-08-02 03:46:27, Tomasz Figa wrote:
>      > +kms_sw_displaytarget_lookup(struct kms_sw_winsys *kms_sw,
>      > +                            unsigned int kms_handle)
> 
>      I don't think the 'lookup' name is appropriate, since it has the
>      ref-count side effect. Instead of 'lookup', how about import, or
>      find_and_import or something similar?
> 
>    How about "get_by_handle"? IMHO "import" is kind of confusing, because the
>    function doesn't end up creating any new objects, but instead just looks
>    through already known ones.

How about find_and_ref?

>    "Get" seems to be a reasonably common word for
>    getting a reference on an object (together with "put" for dropping the
>    reference).
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to