Re: [PATCH v6 0/9] Python bindings for btrace recordings

Doug Evans
2017-02-13T17:18:29Z

Tim Wiederhake <tim.wiederhake@intel.com> writes:
> Hello everyone!
>
> This patch series adds Python bindings for btrace recordings.
>
> V1 of this series can be found here:
> https://sourceware.org/ml/gdb-patches/2016-10/msg00733.html
>
> V2 of this series can be found here:
> https://sourceware.org/ml/gdb-patches/2016-11/msg00084.html
>
> V3 of this series can be found here:
> https://sourceware.org/ml/gdb-patches/2016-11/msg00605.html
>
> V4 of this series can be found here:
> https://sourceware.org/ml/gdb-patches/2017-01/msg00044.html
>
> V5 of this series can be found here:
> https://sourceware.org/ml/gdb-patches/2017-01/msg00616.html
>
> Changes from V5:
> - Skip gdb.python/py-record-full.exp if recording is not supported.
> - s/symbol/SAL/ in comment in py-record-btrace.c.
> - Replaced mem_fileopen by string_file in py-record-btrace.c.
> - Simplified btpy_list_count and added comment in py-record-btrace.c.
> - Removed unused variable in recpy_bt_goto (leftover from previous refactoring)
> in py-record-btrace.c.
> - Moved opening curly braces to new line in py-record-btrace.c.
>
> The documentation parts are already approved by Eli.
>
> Is this good to go?

LGTM