Merge branch 'master' into test_835167838

PiperOrigin-RevId: 835167838

.github/workflows/codeql.yml

@@ -44,7 +44,7 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@cf1bb45a277cb3c205638b2cd5c984db1c46a412 # v3.29.5
+      uses: github/codeql-action/init@e12f0178983d466f2f6028f5cc7a6d786fd97f4b # v3.29.5
       with:
         languages: ${{ matrix.language }}
         # CodeQL is currently crashing on files with large lists:
@@ -56,7 +56,7 @@ jobs:
 
     - if: matrix.language == 'cpp'
       name: Build CPP
-      uses: github/codeql-action/autobuild@cf1bb45a277cb3c205638b2cd5c984db1c46a412 # v3.29.5
+      uses: github/codeql-action/autobuild@e12f0178983d466f2f6028f5cc7a6d786fd97f4b # v3.29.5
 
     - if: matrix.language == 'cpp' || matrix.language == 'java'
       name: Build Java
@@ -66,7 +66,7 @@ jobs:
 
     - if: matrix.language == 'javascript'
       name: Build JS
-      uses: github/codeql-action/autobuild@cf1bb45a277cb3c205638b2cd5c984db1c46a412 # v3.29.5
+      uses: github/codeql-action/autobuild@e12f0178983d466f2f6028f5cc7a6d786fd97f4b # v3.29.5
 
     - if: matrix.language == 'cpp' || matrix.language == 'python'
       name: Build Python
@@ -74,7 +74,7 @@ jobs:
         python setup.py build_ext
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@cf1bb45a277cb3c205638b2cd5c984db1c46a412 # v3.29.5
+      uses: github/codeql-action/analyze@e12f0178983d466f2f6028f5cc7a6d786fd97f4b # v3.29.5
       with:
         category: "/language:${{matrix.language}}"
         ref: "${{ github.ref != 'master' && github.ref || '/refs/heads/master' }}"

.github/workflows/lint.yml

@@ -50,6 +50,6 @@ jobs:
     - name: Lint Python code
       run: |
         eval "$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"
-        ruff check --extend-select=C4,C90,PERF,RET,SIM,W
+        ruff check
 
     # TODO(eustas): run buildifier

.github/workflows/scorecard.yml

@@ -77,6 +77,6 @@ jobs:
 
       # Upload the results to GitHub's code scanning dashboard.
       - name: "Upload to code-scanning"
-        uses: github/codeql-action/upload-sarif@cf1bb45a277cb3c205638b2cd5c984db1c46a412 # v2.23.3
+        uses: github/codeql-action/upload-sarif@e12f0178983d466f2f6028f5cc7a6d786fd97f4b # v2.23.3
         with:
           sarif_file: results.sarif

scripts/dictionary/step-02-rfc-to-bin.py

@@ -18,11 +18,13 @@ for line in lines:
   if appendix_a_found:
     if re_data_line.match(line) is not None:
       data = line.strip()
-      dictionary.extend(int(data[2 * i:2 * i + 2], 16) for i in range(32))
+      for i in range(32):
+        dictionary.append(int(data[2 * i:2 * i + 2], 16))
       if len(dictionary) == 122784:
         break
-  elif line.startswith("Appendix A."):
-    appendix_a_found = True
+  else:
+    if line.startswith("Appendix A."):
+      appendix_a_found = True
 
 bin_path = "dictionary.bin"
 

scripts/dictionary/step-04-generate-java-literals.py

@@ -40,12 +40,13 @@ for b in data:
       is_skip = False
       hi.append(unichr(cntr))
       cntr = skip_flip_offset + 1
-  elif value >= 0x80:
-    cntr += 1
   else:
-    is_skip = True
-    hi.append(unichr(cntr))
-    cntr = skip_flip_offset + 1
+    if value >= 0x80:
+      cntr += 1
+    else:
+      is_skip = True
+      hi.append(unichr(cntr))
+      cntr = skip_flip_offset + 1
 hi.append(unichr(cntr))
 
 low0 = low[0:len(low) // 2]
@@ -55,15 +56,15 @@ low1 = low[len(low) // 2:len(low)]
 def escape(chars):
   result = []
   for c in chars:
-    if c == "\r":
+    if "\r" == c:
       result.append("\\r")
-    elif c == "\n":
+    elif "\n" == c:
       result.append("\\n")
-    elif c == "\t":
+    elif "\t" == c:
       result.append("\\t")
-    elif c == "\"":
+    elif "\"" == c:
       result.append("\\\"")
-    elif c == "\\":
+    elif "\\" == c:
       result.append("\\\\")
     elif ord(c) < 32 or ord(c) >= 127:
       result.append("\\u%04X" % ord(c))

setup.py

@@ -53,7 +53,8 @@ def get_version():
 
 def get_test_suite():
     test_loader = unittest.TestLoader()
-    return test_loader.discover("python", pattern="*_test.py")
+    test_suite = test_loader.discover("python", pattern="*_test.py")
+    return test_suite
 
 
 class BuildExt(build_ext):
@@ -76,9 +77,13 @@ class BuildExt(build_ext):
         if not (self.force or dep_util.newer_group(depends, ext_path, "newer")):
             log.debug("skipping '%s' extension (up-to-date)", ext.name)
             return
-        log.info("building '%s' extension", ext.name)
+        else:
+            log.info("building '%s' extension", ext.name)
 
-        c_sources = [source for source in ext.sources if source.endswith(".c")]
+        c_sources = []
+        for source in ext.sources:
+            if source.endswith(".c"):
+                c_sources.append(source)
         extra_args = ext.extra_compile_args or []
 
         objects = []
@@ -179,7 +184,7 @@ USE_SYSTEM_BROTLI = bool_from_environ("USE_SYSTEM_BROTLI")
 
 if USE_SYSTEM_BROTLI:
     import pkgconfig
-
+    
     REQUIRED_BROTLI_SYSTEM_LIBRARIES = ["libbrotlicommon", "libbrotlienc", "libbrotlidec"]
 
     define_macros = []