On Tue, May 5, 2015 at 11:25 AM, Neil Roberts <n...@linux.intel.com> wrote:
> Jason Ekstrand <ja...@jlekstrand.net> writes:
>
>> +static inline void list_validate(struct list_head *list)
>> +{
>> + assert(list->next->prev == list && list->prev->next == list);
>> + for (struct list_head *node = list->next; node != list; node =
>> node->next)
>> + assert(node->next->prev == node && node->prev->next == node);
>> +}
>
> It would be good if this #included assert.h. It might be nice to use one of
> the iterator macros too, although I guess that's just bikeshedding so
> I won't complain either way.
Rob pointed that out too. Fixed locally.
--Jason
> Regards,
> - Neil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
