<p>osmith has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ci/+/24867">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lint: ignore COMPLEX_MACRO<br><br>Complains about, for example:<br>  #define MDCX4 \<br>        "MDCX 18983217 1@mgw MGCP 1.0\r\n" \<br>        "M: sendrecv\r" \<br>   "C: 2\r\n" \<br>        "I: %s\r\n" \<br>       "L: p:20, a:AMR, nt:IN\r\n" \<br>       "\n" \<br>      "v=0\r\n" \<br> "o=- %s 23 IN IP4 5.6.7.8\r\n" \<br>    "c=IN IP4 5.6.7.8\r\n" \<br>    "t=0 0\r\n" \<br>       "m=audio 4441 RTP/AVP 99\r\n" \<br>     "a=rtpmap:99 AMR/8000\r\n" \<br>        "a=ptime:40\r\n"<br><br>Related: OS#5087<br>Change-Id: Ic9d752ca841161a62e3631c84b4237a0d8594363<br>---<br>M lint/checkpatch/checkpatch_osmo.sh<br>1 file changed, 2 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ci refs/changes/67/24867/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh</span><br><span>index 208e8b6..630fa86 100755</span><br><span>--- a/lint/checkpatch/checkpatch_osmo.sh</span><br><span>+++ b/lint/checkpatch/checkpatch_osmo.sh</span><br><span>@@ -9,6 +9,7 @@</span><br><span> # * ASSIGN_IN_IF: not followed (e.g. 'if ((u8 = gsup_msg->cause))')</span><br><span> # * AVOID_EXTERNS: we do use externs</span><br><span> # * BLOCK_COMMENT_STYLE: we don't use a trailing */ on a separate line</span><br><span style="color: hsl(120, 100%, 40%);">+# * COMPLEX_MACRO: we don't use parentheses when building macros of strings across multiple lines</span><br><span> # * CONSTANT_COMPARISON: not followed: "Comparisons should place the constant on the right side"</span><br><span> # * EMBEDDED_FUNCTION_NAME: often __func__ isn't used, arguably not much benefit in changing this when touching code</span><br><span> # * EXECUTE_PERMISSIONS: not followed, files need to be executable: git-version-gen, some in debian/</span><br><span>@@ -29,6 +30,7 @@</span><br><span>         --ignore ASSIGN_IN_IF \</span><br><span>      --ignore AVOID_EXTERNS \</span><br><span>     --ignore BLOCK_COMMENT_STYLE \</span><br><span style="color: hsl(120, 100%, 40%);">+        --ignore COMPLEX_MACRO \</span><br><span>     --ignore CONSTANT_COMPARISON \</span><br><span>       --ignore EMBEDDED_FUNCTION_NAME \</span><br><span>    --ignore EXECUTE_PERMISSIONS \</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ci/+/24867">change 24867</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/c/osmo-ci/+/24867"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ci </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic9d752ca841161a62e3631c84b4237a0d8594363 </div>
<div style="display:none"> Gerrit-Change-Number: 24867 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>