diff options
author | Yaniv Kaul <ykaul@redhat.com> | 2019-03-08 14:13:32 +0200 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2019-04-25 04:13:31 +0000 |
commit | 3db4f4ed6dec55316e7167eaddd9ddf0af875185 (patch) | |
tree | 2a2381befb39488755d207dfbd848c176524eda7 /build-aux | |
parent | 1ccc62d11287e8ffd091123517f7988a4e7f1935 (diff) | |
download | glusterfs-3db4f4ed6dec55316e7167eaddd9ddf0af875185.tar.gz glusterfs-3db4f4ed6dec55316e7167eaddd9ddf0af875185.tar.xz glusterfs-3db4f4ed6dec55316e7167eaddd9ddf0af875185.zip |
logging.c/h: aggressively remove sprintfs()
Try to reduce the number of sprintf() and string copies until we finally
log a log line.
Specifically, do not sprintf separately the timestr string and
do not sprintf/strcpy the appmsgstr separately - just stick it with
the header.
Hoping I did not leak anything or changed the log line formatting.
Also, allocate 4K (GF_LOG_BACKTRACE_SIZE) of memory
dynamically for trace output -
only if trace was actually requested (previously, it was
unconditionally)
In addition, some minor code formatting (unrelated to the above).
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Change-Id: Id2ccc85f9213a2b1c6eaa4a2f58ce043eac1824f
Diffstat (limited to 'build-aux')
0 files changed, 0 insertions, 0 deletions