[Findutils-patches] [PATCH] Use set_program_name in regexprops.

[James Youngman]

* lib/regexprops.c: Include "progname.h".  Update function
defintions to have a newline before the function name (to comply
with the GNU coding standard).  Use set_program_name.
---
 ChangeLog        |    7 ++++++
 lib/regexprops.c |   61 ++++++++++++++++++++++++++++++++++-------------------
 2 files changed, 46 insertions(+), 22 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index ae1ff04..d49e0a1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-04-05  James Youngman  <j...@gnu.org>
+
+       Use set_program_name in regexprops.
+       * lib/regexprops.c: Include "progname.h".  Update function
+       defintions to have a newline before the function name (to comply
+       with the GNU coding standard).  Use set_program_name.
+
 2010-04-05  Jim Meyering  <meyer...@redhat.com>
 
        frcode: avoid link failure due to multiple program_name definition
diff --git a/lib/regexprops.c b/lib/regexprops.c
index 7facd0d..efef3ec 100644
--- a/lib/regexprops.c
+++ b/lib/regexprops.c
@@ -36,12 +36,11 @@
 
 #include "regex.h"
 #include "regextype.h"
+#include "progname.h"
 
 
-/* Name this program was run with. */
-char *program_name;
-
-static void output (const char *s, int escape)
+static void
+output (const char *s, int escape)
 {
   (void) escape;
 
@@ -49,34 +48,40 @@ static void output (const char *s, int escape)
 }
 
 
-static void newline (void)
+static void
+newline (void)
 {
   output ("\n", 0);
 }
 
-static void content (const char *s)
+static void
+content (const char *s)
 {
   output (s, 1);
 }
 
-static void literal (const char *s)
+static void
+literal (const char *s)
 {
   output (s, 0);
 }
 
-static void directive (const char *s)
+static void
+directive (const char *s)
 {
   output (s, 0);
 }
 
-static void comment (const char *s)
+static void
+comment (const char *s)
 {
   directive ("@c ");
   literal (s);
   newline ();
 }
 
-static void enum_item (const char *s)
+static void
+enum_item (const char *s)
 {
   newline ();
   directive ("@item ");
@@ -84,10 +89,11 @@ static void enum_item (const char *s)
   newline ();
 }
 
-static void begin_subsection (const char *name,
-                             const char *next,
-                             const char *prev,
-                             const char *up)
+static void
+begin_subsection (const char *name,
+                 const char *next,
+                 const char *prev,
+                 const char *up)
 {
   (void) next;
   (void) prev;
@@ -108,7 +114,8 @@ static void begin_subsection (const char *name,
   newline ();
 }
 
-static void begintable_markup (char const *markup)
+static void
+begintable_markup (char const *markup)
 {
   newline ();
   directive ("@table ");
@@ -116,28 +123,32 @@ static void begintable_markup (char const *markup)
   newline ();
 }
 
-static void endtable ()
+static void
+endtable ()
 {
   newline ();
   directive ("@end table");
   newline ();
 }
 
-static void beginenum ()
+static void
+beginenum ()
 {
   newline ();
   directive ("@enumerate");
   newline ();
 }
 
-static void endenum ()
+static void
+endenum ()
 {
   newline ();
   directive ("@end enumerate");
   newline ();
 }
 
-static void newpara ()
+static void
+newpara ()
 {
   content ("\n\n");
 }
@@ -434,7 +445,8 @@ describe_regex_syntax (int options)
 }
 
 
-static void copying (void)
+static void
+copying (void)
 {
   static const char *copy_para[]=
     {
@@ -561,12 +573,17 @@ describe_all (const char *contextname,
 
 
 
-int main (int argc, char *argv[])
+int
+main (int argc, char *argv[])
 {
   const char *up = "";
   unsigned int context = CONTEXT_ALL;
   const char *contextname = "all";
-  program_name = argv[0];
+
+  if (argc)
+    set_program_name (argv[0]);
+  else
+    set_program_name ("regexprops");
 
   if (argc > 1)
     {
-- 
1.7.0



_______________________________________________
Findutils-patches mailing list
Findutils-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/findutils-patches