krb5 commit: Fix Python regexp literals

ghudson at mit.edu ghudson at mit.edu
Wed May 22 17:26:31 EDT 2024


https://github.com/krb5/krb5/commit/4b21b2e2821d3cb91042be09e0ebe09707a57d72
commit 4b21b2e2821d3cb91042be09e0ebe09707a57d72
Author: Arjun <pkillarjun at protonmail.com>
Date:   Thu May 9 20:47:08 2024 +0530

    Fix Python regexp literals
    
    Add missing "r" prefixes before literals using regexp escape
    sequences.
    
    [ghudson at mit.edu: split into separate commit; rewrote commit message]

 src/util/cstyle-file.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/util/cstyle-file.py b/src/util/cstyle-file.py
index 837fa05e1..56b1e32a8 100644
--- a/src/util/cstyle-file.py
+++ b/src/util/cstyle-file.py
@@ -208,7 +208,7 @@ def check_assignment_in_conditional(line, ln):
 
 
 def indent(line):
-    return len(re.match('\s*', line).group(0).expandtabs())
+    return len(re.match(r'\s*', line).group(0).expandtabs())
 
 
 def check_unbraced_flow_body(line, ln, lines):
@@ -220,8 +220,8 @@ def check_unbraced_flow_body(line, ln, lines):
     if m and (m.group(1) is None) != (m.group(3) is None):
         warn(ln, 'One arm of if/else statement braced but not the other')
 
-    if (re.match('\s*(if|else if|for|while)\s*\(.*\)$', line) or
-        re.match('\s*else$', line)):
+    if (re.match(r'\s*(if|else if|for|while)\s*\(.*\)$', line) or
+        re.match(r'\s*else$', line)):
         base = indent(line)
         # Look at the next two lines (ln is 1-based so lines[ln] is next).
         if indent(lines[ln]) > base and indent(lines[ln + 1]) > base:


More information about the cvs-krb5 mailing list