[waffle] [PATCH 00/13] Core waffle cleanups

Tapani Pälli tapani.palli at intel.com
Tue Aug 23 09:52:04 UTC 2016



On 08/18/2016 01:28 PM, Emil Velikov wrote:
> On 21 June 2016 at 11:33, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>> On 16 May 2016 at 11:57, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>> On 16 May 2016 at 11:54, Emil Velikov <emil.l.velikov at gmail.com> wrote:
>>>> Hi all,
>>>>
>>>> While looking at the gbm/egl I've noticed a few interesting bits.
>>>>  - We do NULL checking for values that are guaranteed by API to be
>>>> non-NULL.
>>>>  - wcore_*_init does not need a return type, plus in some places we were
>>>> not calling it in the correct time.
>>>>  - wcore_*_teardown is a simple wrapper around assert, which (at the
>>>> time the function should be called) is too late/not needed.
>>>>
>>>> So this series simplifies these, giving us a nice -350 line count ;-)
>>>>
>>>> The whole thing can be found in
>>>> https://github.com/evelikov/waffle/tree/for-upstream/core-cleanups
>>>>
>>> For some reason git send-email seems to be choking on patches 08/13
>>> and 09/13. Please check those out via the above repo or let me know if
>>> you'd prefer them in other format.
>>>
>> I might have gone overboard (too much) folding the error label(s) in
>> 09/13 "core: remove wcore_*_init() return type". I can split those up
>> if people prefer.
>>
> Humble poke.


Patches 1 (cleanup) and 3-7 (do not check null since api_check_entry did 
it already):

Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

(I will check the rest from the repo)

> Emil
> _______________________________________________
> waffle mailing list
> waffle at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/waffle
>


More information about the waffle mailing list