printf() WARN fix for %argnum$ > 9

peachpiecompiler · Aug 16, 2024 · fec3124 · fec3124
1 parent 4980fd9
commit fec3124
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Peachpie.CodeAnalysis/FlowAnalysis/Passes/DiagnosticWalker.Functions.cs b/src/Peachpie.CodeAnalysis/FlowAnalysis/Passes/DiagnosticWalker.Functions.cs
@@ -22,7 +22,7 @@ internal partial class DiagnosticWalker<T>
         /// Matches <c>printf()</c> format specifier.
         /// </summary>
         static readonly Lazy<Regex> s_printfSpecsRegex = new Lazy<Regex>(
-            () => new Regex(@"%(?:(\d)+\$)?[+-]?(?:[ 0]|'.{1})?-?\d*(?:\.\d+)?[bcdeEufFgGosxX]", RegexOptions.Compiled | RegexOptions.CultureInvariant)
+            () => new Regex(@"%(?:(\d+)\$)?[+-]?(?:[ 0]|'.{1})?-?\d*(?:\.\d+)?[bcdeEufFgGosxX]", RegexOptions.Compiled | RegexOptions.CultureInvariant)
         );
 
         void printfCheck(string name, ImmutableArray<BoundArgument> arguments)
@@ -44,7 +44,7 @@ void printfCheck(string name, ImmutableArray<BoundArgument> arguments)
                     else
                     {
                         // %2$d
-                        int numSpec = int.Parse(numSpecStr);
+                        int numSpec = int.Parse(numSpecStr); // > 0, starts from 1
                         numSpecMax = Math.Max(numSpec, numSpecMax);
                     }
                 }