[libav-devel] [GASPP PATCH 1/2] Fix the regexp used for replacing .align

Martin Storsjo martin at martin.st
Fri Oct 19 23:07:51 CEST 2018


The condition above allows multiple spaces, but the actual replacement
only allowed one space.
---
 gas-preprocessor.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
index c56301c..b22ee8a 100755
--- a/gas-preprocessor.pl
+++ b/gas-preprocessor.pl
@@ -922,7 +922,7 @@ sub handle_serialized_line {
         # ALIGN in armasm syntax is the actual number of bytes
         if ($line =~ /\.(?:p2)?align\s+(\d+)/) {
             my $align = 1 << $1;
-            $line =~ s/\.(?:p2)?align\s(\d+)/ALIGN $align/;
+            $line =~ s/\.(?:p2)?align\s+(\d+)/ALIGN $align/;
         }
         # Convert gas style [r0, :128] into armasm [r0 at 128] alignment specification
         $line =~ s/\[([^\[,]+),?\s*:(\d+)\]/[$1\@$2]/g;
-- 
2.7.4



More information about the libav-devel mailing list