On 12/21/2020 4:49 PM, Daniel Stenberg via curl-library wrote:
On Mon, 21 Dec 2020, Dennis Clarke via curl-library wrote:

Ah ha, well that provides endless buckets of information.

It seems the script doesn't properly ignore lines with nothing but a single hash ('#') sign on them. My proposed patch:

diff --git a/tests/badsymbols.pl b/tests/badsymbols.pl
index bad037975..e609633bb 100755
--- a/tests/badsymbols.pl
+++ b/tests/badsymbols.pl
@@ -88,10 +88,13 @@ sub scanenums {
             next;
         }
         if($skipit) {
             next;
         }
+        if (/^\#\s*\z/) {
+            next;
+        }
         if ( /enum\s+(\S+\s+)?{/ .. /}/ ) {
             s/^\s+//;
             chomp;
             s/[,\s].*//;
             if(($_ !~ /\}(;|)/) &&



Daniel has submitted his patch as a PR [1] and I will follow up there with comments.



[1]: https://github.com/curl/curl/pull/6355


-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:   https://curl.se/mail/etiquette.html

Reply via email to