[lttng-dev] [PATCH lttng-tools v2] Bug 1002

[Jonathan Rajotte]

Hi Mathieu,

The following patches should make these changes more explicit. Thanks for
pointing that out.

The main reason for the refactoring is that bug 1002[1] is caused by
the presence of 2 code paths for snapshot recording: one for mi and one for
normal execution. Since MSG macro are disabled with the presence of --mi on the
command line, it is simpler to embed the mi code as done in other commands.

Bug 1002[1] also mention that a simple warning when attempting an empty snapshot
record should not return an error to the command line. This is fixed in
"Fix: do not return error on LTTNG_ERR_SNAPSHOT_NODATA".

[1] https://bugs.lttng.org/issues/1002

Cheers