[MERGED] libosmocore[master]: utils: add osmo_escape_str()

2017-12-18 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: utils: add osmo_escape_str() .. utils: add osmo_escape_str() To report invalid characters in identifiers, it is desirable to escape any weird

libosmocore[master]: utils: add osmo_escape_str()

2017-12-18 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/5429 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic685eb63dead3967d01aaa4f1e9899e5461ca49a Gerrit-PatchSet: 3 Gerrit-Project: libosmocore Gerrit-Branch: master

[PATCH] libosmocore[master]: utils: add osmo_escape_str()

2017-12-17 Thread Neels Hofmeyr
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/5429 to look at the new patch set (#3). utils: add osmo_escape_str() To report invalid characters in identifiers, it is desirable to escape any weird characters. Otherwise we might print

libosmocore[master]: utils: add osmo_escape_str()

2017-12-16 Thread Harald Welte
Patch Set 2: (1 comment) https://gerrit.osmocom.org/#/c/5429/2/src/utils.c File src/utils.c: Line 498: whitespace -- To view, visit https://gerrit.osmocom.org/5429 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id:

[PATCH] libosmocore[master]: utils: add osmo_escape_str()

2017-12-15 Thread Neels Hofmeyr
Review at https://gerrit.osmocom.org/5429 utils: add osmo_escape_str() To report invalid characters in identifiers, it is desirable to escape any weird characters. Otherwise we might print stray newlines or control characters in the log output. ctrl_test.c already uses a print_escaped()