diff options
author | Denys Vlasenko <dvlasenk@redhat.com> | 2010-09-14 14:05:46 +0200 |
---|---|---|
committer | Denys Vlasenko <dvlasenk@redhat.com> | 2010-09-14 14:05:46 +0200 |
commit | fed72f316d52c65ee1aea9f4b36dac19adb6349a (patch) | |
tree | 2cb6c356511c44e30819a1ff2933d5de1c58a391 /lib/utils/logging.c | |
parent | 612234fe9c42ddbb9a91f1f247f2b8daff93f4d8 (diff) | |
download | abrt-fed72f316d52c65ee1aea9f4b36dac19adb6349a.tar.gz abrt-fed72f316d52c65ee1aea9f4b36dac19adb6349a.tar.xz abrt-fed72f316d52c65ee1aea9f4b36dac19adb6349a.zip |
msg_prefix now doesn't need ": " suffix
Contains some other trivial changes
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
Diffstat (limited to 'lib/utils/logging.c')
-rw-r--r-- | lib/utils/logging.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/lib/utils/logging.c b/lib/utils/logging.c index b552fa8a..815efb5e 100644 --- a/lib/utils/logging.c +++ b/lib/utils/logging.c @@ -55,16 +55,19 @@ static void verror_msg_helper(const char *s, * This is needed for e.g. when multiple children * can produce log messages simultaneously. */ - prefix_len = strlen(msg_prefix); + prefix_len = msg_prefix[0] ? strlen(msg_prefix) + 2 : 0; strerr_len = strerr ? strlen(strerr) : 0; msgeol_len = strlen(msg_eol); /* +3 is for ": " before strerr and for terminating NUL */ msg = (char*) xrealloc(msg, prefix_len + used + strerr_len + msgeol_len + 3); /* TODO: maybe use writev instead of memmoving? Need full_writev? */ if (prefix_len) { + char *p; memmove(msg + prefix_len, msg, used); used += prefix_len; - memcpy(msg, msg_prefix, prefix_len); + p = stpcpy(msg, msg_prefix); + p[0] = ':'; + p[1] = ' '; } if (strerr) { if (s[0]) { |