laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
logging: fix coding style issues in _output_buf()

Change-Id: Ia794dbf51045361ee40815f708344c88abe87a96
---
M src/logging.c
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/logging.c b/src/logging.c
index 148bb27..5336e53 100644
--- a/src/logging.c
+++ b/src/logging.c
@@ -580,19 +580,19 @@
* message in '\n'. If so, nip the last '\n' away, insert the source file info and re-append an
* '\n'. All this to allow LOGP("start..."); LOGPC("...end\n") constructs. */
if (target->print_filename_pos == LOG_FILENAME_POS_LINE_END
- && offset > 0 && buf[offset-1] == '\n') {
+ && offset > 0 && buf[offset - 1] == '\n') {
switch (target->print_filename2) {
case LOG_FILENAME_NONE:
break;
case LOG_FILENAME_PATH:
- offset --;
+ offset--;
ret = snprintf(buf + offset, rem, " (%s:%d)\n", file, line);
if (ret < 0)
goto err;
OSMO_SNPRINTF_RET(ret, rem, offset, len);
break;
case LOG_FILENAME_BASENAME:
- offset --;
+ offset--;
ret = snprintf(buf + offset, rem, " (%s:%d)\n", const_basename(file), line);
if (ret < 0)
goto err;

3 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

To view, visit change 27011. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia794dbf51045361ee40815f708344c88abe87a96
Gerrit-Change-Number: 27011
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged