Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 8707] Preferences with trailing commas are ignored

[bugzilla-daemon@xxxxxxxxxxxxx]

Comment # 6 on bug 8707 from Cal Turney

(In reply to comment #5)
>            if (cur_var->len > 0) {
             if (got_val) {
+            
> if (cur_val->str[strlen(cur_val->str)-1] == ',') {
+                /* 
+   
> * If the pref has a trailing comma, eliminate it.
+                 */
+    
> cur_val->str[strlen(cur_val->str)-1] = '\0';

can we use cur_var->len
> instead of strlen() ?

Good idea.  I'll resubmit the patch shortly.  Thanks.

---

      You are receiving this mail because:

• You are watching all bug changes.