[Libreoffice] Assertions and Logging

Stephan Bergmann sbergman at redhat.com
Wed Nov 23 08:12:38 PST 2011


On 11/23/2011 05:04 PM, Norbert Thiebaud wrote:
> On Wed, Nov 23, 2011 at 9:24 AM, Stephan Bergmann<sbergman at redhat.com>  wrote:
> who said that sal has to auto-initialized the service.
>
> main() explicitly call sal_init_trace_services(char*
> log_env_descrition) (or whatever) and store the resulting array of
> logging context in a globally visible variable that the tracing macro
> can use.

sure; see my "though by adding complexity" parenthetical remark below

>> Honestly, I am not worried about the performance of the current
>> implementation at all.  If it should ever become a bottleneck (esp. if we
>> ever decide to enable logs in production builds), we could rather easily
>> improve it (though by adding complexity), without (that's my hope having
>> designed it that way) having to modify the interface.
>
> If you use a string as a trace-selector you will never get something
> with performance good-enough for release code.
> you need a numeric level and a numeric module/feature selector.
>
> You need a system so that when the trace are not wanted a trace-point
> cost a couple of integer testing at worse. you can't take a call, you
> cant' have to construct arguments for the call,
> you certainly can't have c++ object instantiation/clone/copy etc...

numbers, please  :)

Stephan


More information about the LibreOffice mailing list