[lttng-dev] [BABELTRACE PATCH] Use default if there is no clock declaration

[Brosseau, Yannick]

On Mon, Feb 13, 2012 at 9:26 AM, Mathieu Desnoyers
<mathieu.desnoyers at efficios.com> wrote:
> * Mathieu Desnoyers (mathieu.desnoyers at efficios.com) wrote:
>> * Juha Niskanen (juniskane at gmail.com) wrote:
>> > In absence of clock description in metadata, use a default clock
>> > source which increments once per nanosecond.
>> >
>> > Without this patch, the converter crashes when given old traces
>> > that are lacking clock description block.
>>
>> Merged (with tiny cosmetic fixes). Thanks !
>
> FYI, with commit:
>
> commit 02dc4610b1dccefaf545ba77378857ee6188ea33
> Author: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
> Date:   Mon Feb 13 09:03:55 2012 -0500
>
>    lib: open trace error handling fix
>
>    When a trace from a trace collection path fails, consider this as a hard
>    error. Print errors to stderr rather than stdout.
>
>    The end goal is to never show an output if any of the trace specified
>    (or trace within a trace collection) fails to open.
>
>    Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers at efficios.com>
>

Should you remove the traces that were added to the context before you
found an error?