<p>laforge has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/simtrace2/+/16635">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">simtrace2-sniff: Reformat value_string to pass our validation scripts<br><br>This is a purely cosmetic change in terms of coding style, but it<br>fixes an [invalid] detection for unterminated value_string arrays<br>by our verify_value_string_arrays_are_terminated.py script:<br><br>Change-Id: Iee905c744065df1dc54f04be752a65700897a92c<br>ERROR: file contains unterminated value_string 'struct value_string change_flags[]': './deps/simtrace2/host/src/simtrace2-sniff.c'<br>ERROR: file contains unterminated value_string 'struct value_string data_flags[]': './deps/simtrace2/host/src/simtrace2-sniff.c'<br>---<br>M host/src/simtrace2-sniff.c<br>1 file changed, 10 insertions(+), 40 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/simtrace2 refs/changes/35/16635/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/host/src/simtrace2-sniff.c b/host/src/simtrace2-sniff.c</span><br><span>index b2ce4a0..c4cec1e 100644</span><br><span>--- a/host/src/simtrace2-sniff.c</span><br><span>+++ b/host/src/simtrace2-sniff.c</span><br><span>@@ -61,49 +61,19 @@</span><br><span> };</span><br><span> </span><br><span> const struct value_string change_flags[] = {</span><br><span style="color: hsl(0, 100%, 40%);">-        {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_CHANGE_FLAG_CARD_INSERT,</span><br><span style="color: hsl(0, 100%, 40%);">-         .str = "card inserted",</span><br><span style="color: hsl(0, 100%, 40%);">-       },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_CHANGE_FLAG_CARD_EJECT,</span><br><span style="color: hsl(0, 100%, 40%);">-          .str = "card ejected",</span><br><span style="color: hsl(0, 100%, 40%);">-        },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_CHANGE_FLAG_RESET_ASSERT,</span><br><span style="color: hsl(0, 100%, 40%);">-                .str = "reset asserted",</span><br><span style="color: hsl(0, 100%, 40%);">-      },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_CHANGE_FLAG_RESET_DEASSERT,</span><br><span style="color: hsl(0, 100%, 40%);">-              .str = "reset de-asserted",</span><br><span style="color: hsl(0, 100%, 40%);">-   },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_CHANGE_FLAG_TIMEOUT_WT,</span><br><span style="color: hsl(0, 100%, 40%);">-          .str = "data transfer timeout",</span><br><span style="color: hsl(0, 100%, 40%);">-       },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = 0,</span><br><span style="color: hsl(0, 100%, 40%);">-             .str = NULL,</span><br><span style="color: hsl(0, 100%, 40%);">-    },</span><br><span style="color: hsl(120, 100%, 40%);">+    { SNIFF_CHANGE_FLAG_CARD_INSERT, "card inserted" },</span><br><span style="color: hsl(120, 100%, 40%);">+ { SNIFF_CHANGE_FLAG_CARD_EJECT, "card ejected" },</span><br><span style="color: hsl(120, 100%, 40%);">+   { SNIFF_CHANGE_FLAG_RESET_ASSERT, "reset asserted" },</span><br><span style="color: hsl(120, 100%, 40%);">+       { SNIFF_CHANGE_FLAG_RESET_DEASSERT, "reset de-asserted" },</span><br><span style="color: hsl(120, 100%, 40%);">+  { SNIFF_CHANGE_FLAG_TIMEOUT_WT, "data transfer timeout" },</span><br><span style="color: hsl(120, 100%, 40%);">+  { 0, NULL }</span><br><span> };</span><br><span> </span><br><span> const struct value_string data_flags[] = {</span><br><span style="color: hsl(0, 100%, 40%);">-     {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_DATA_FLAG_ERROR_INCOMPLETE,</span><br><span style="color: hsl(0, 100%, 40%);">-              .str = "incomplete",</span><br><span style="color: hsl(0, 100%, 40%);">-  },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_DATA_FLAG_ERROR_MALFORMED,</span><br><span style="color: hsl(0, 100%, 40%);">-               .str = "malformed",</span><br><span style="color: hsl(0, 100%, 40%);">-   },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = SNIFF_DATA_FLAG_ERROR_CHECKSUM,</span><br><span style="color: hsl(0, 100%, 40%);">-                .str = "checksum error",</span><br><span style="color: hsl(0, 100%, 40%);">-      },</span><br><span style="color: hsl(0, 100%, 40%);">-      {</span><br><span style="color: hsl(0, 100%, 40%);">-               .value = 0,</span><br><span style="color: hsl(0, 100%, 40%);">-             .str = NULL,</span><br><span style="color: hsl(0, 100%, 40%);">-    },</span><br><span style="color: hsl(120, 100%, 40%);">+    { SNIFF_DATA_FLAG_ERROR_INCOMPLETE, "incomplete" },</span><br><span style="color: hsl(120, 100%, 40%);">+ { SNIFF_DATA_FLAG_ERROR_MALFORMED, "malformed" },</span><br><span style="color: hsl(120, 100%, 40%);">+   { SNIFF_DATA_FLAG_ERROR_CHECKSUM, "checksum error" },</span><br><span style="color: hsl(120, 100%, 40%);">+       { 0, NULL }</span><br><span> };</span><br><span> </span><br><span> static void print_flags(const struct value_string* flag_meanings, uint32_t nb_flags, uint32_t flags) {</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/simtrace2/+/16635">change 16635</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/simtrace2/+/16635"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: simtrace2 </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iee905c744065df1dc54f04be752a65700897a92c </div>
<div style="display:none"> Gerrit-Change-Number: 16635 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>