On 11/23/2011 05:04 PM, Norbert Thiebaud wrote:
On Wed, Nov 23, 2011 at 9:24 AM, Stephan Bergmann<sbergman@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
Context
- Re: [Libreoffice] Assertions and Logging (continued)
Re: [Libreoffice] Assertions and Logging · Stephan Bergmann
Re: [Libreoffice] Assertions and Logging · Bjoern Michaelsen
Re: [Libreoffice] Assertions and Logging · Michael Stahl
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.