On 06/10/2014 11:00 AM, Matt Turner wrote: > Our most frequently used list implementation was only usable from > C by manipulating the members of the structs. This series adds a > C API, wraps the C API with the C++ member functions (so no change > for C++ code), makes the foreach_list macros usable from C, and > replaces an open-coded list in the i965 driver with exec_list. > > The C API consists of static inline functions, and the C++ member > functions are also marked inline.
Before seeing this series, I would have sworn up and down that at least the core list API as available from C. It really did start life as a C API, but in the C++ switch the C version must have been dropped for "clarity." Assuming the answer to my question in patch 7 is, "No new warnings," the series is Reviewed-by: Ian Romanick <ian.d.roman...@intel.com> > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
