<p style="white-space: pre-wrap; word-wrap: break-word;">Change looks good to me.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I find this useful, and I can't see how having a multiline string logged here would be an issue. As Max said, it is only logged in case of error anyway. The code does look cleaner to me in the new version.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Old output:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> Mismatch! Expected:<br> 14010809710000004026f0280102</pre><p style="white-space: pre-wrap; word-wrap: break-word;">New output:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> DLGLOBAL __wrap_osmo_gsup_client_send() L0 data mismatch:<br> expected 14 01 08 09 71 00 00 00 40 26 f0 28 01 02 <br>          !!<br>     msgb 04 01 08 09 71 00 00 00 40 26 f0 28 01 02</pre><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/12571">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12571">change 12571</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/12571"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If9771c973f2bc55580f4c146bdbeeb1609d56786 </div>
<div style="display:none"> Gerrit-Change-Number: 12571 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 24 Jan 2019 17:15:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>