laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/27011 )

Change subject: logging: fix coding style issues in _output_buf()
......................................................................

logging: fix coding style issues in _output_buf()

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

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



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 https://gerrit.osmocom.org/c/libosmocore/+/27011
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia794dbf51045361ee40815f708344c88abe87a96
Gerrit-Change-Number: 27011
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged

Reply via email to