[svn:parrot] r37880 - in trunk: lib/Parrot t/codingstd tools/build

[coke at svn.parrot.org]

Author: coke
Date: Fri Apr  3 00:00:24 2009
New Revision: 37880
URL: https://trac.parrot.org/parrot/changeset/37880

Log:
[t/docs] Fix a bug in headerizer (and its duplicated method) that extracted the non-function declaration '}'

This gets another file passing function_docs (no longer reporting non-existant failures)

Modified:
   trunk/lib/Parrot/Headerizer.pm
   trunk/t/codingstd/c_function_docs.t
   trunk/tools/build/headerizer.pl

Modified: trunk/lib/Parrot/Headerizer.pm
==============================================================================
--- trunk/lib/Parrot/Headerizer.pm	Thu Apr  2 20:27:20 2009	(r37879)
+++ trunk/lib/Parrot/Headerizer.pm	Fri Apr  3 00:00:24 2009	(r37880)
@@ -119,6 +119,9 @@
     # If it's got a semicolon, it's not a function header
     @funcs = grep { !/;/ } @funcs;
 
+    # remove any remaining }'s
+    @funcs = grep {! /^}/} @funcs;
+
     chomp @funcs;
 
     return @funcs;

Modified: trunk/t/codingstd/c_function_docs.t
==============================================================================
--- trunk/t/codingstd/c_function_docs.t	Thu Apr  2 20:27:20 2009	(r37879)
+++ trunk/t/codingstd/c_function_docs.t	Fri Apr  3 00:00:24 2009	(r37880)
@@ -103,7 +103,6 @@
 compilers/pirc/src/piremit.c
 compilers/pirc/src/pirerr.c
 compilers/pirc/src/pirmacro.c
-compilers/pirc/src/pirop.c
 compilers/pirc/src/pirpcc.c
 compilers/pirc/src/pirregalloc.c
 compilers/pirc/src/pirsymbol.c

Modified: trunk/tools/build/headerizer.pl
==============================================================================
--- trunk/tools/build/headerizer.pl	Thu Apr  2 20:27:20 2009	(r37879)
+++ trunk/tools/build/headerizer.pl	Fri Apr  3 00:00:24 2009	(r37880)
@@ -144,6 +144,9 @@
     # If it's got a semicolon, it's not a function header
     @funcs = grep !/;/, @funcs;
 
+    # remove any remaining }'s
+    @funcs = grep {! /^}/} @funcs;
+
     chomp @funcs;
 
     return @funcs;