RE: [PATCH 2/3] btrace: Remove VEC cleanups

Metzger, Markus T
2018-03-05T12:39:23Z

Hello Simon,

> This patch replaces two VEC(tp_t) with std::vector<thread_info *>, which
> allows to remove two cleanups. To make it easier to map the old code to
> the new code, I added the ordered_remove and unordered_remove functions,
> which operate on std::vector and do the same as VEC's
> ordered_remove/unordered_remove.

Looks good to me.

Regards,
Markus.
Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Christian Lamprechter
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928