cppmain.c (do_preprocessing): New function; most of the old main.

        * cppmain.c (do_preprocessing): New function; most of the old
        main.
        (main): Call it to do most of the work.
        (cb): Move from global scope to set_callbacks ().
        (setup_callbacks): Get the callback pointer.
        (general_init, printer_init): Clean up code and comments.

From-SVN: r39012

gcc/ChangeLog

+2001-01-14  Neil Booth  <neil@daikokuya.demon.co.uk>
+
+        * cppmain.c (do_preprocessing): New function; most of the old
+        main.
+        (main): Call it to do most of the work.
+        (cb): Move from global scope to set_callbacks ().
+        (setup_callbacks): Get the callback pointer.
+        (general_init, printer_init): Clean up code and comments.
+
 2001-01-14  Richard Earnshaw  <rearnsha@arm.com>
 
 	* config/arm/semi.h (ASM_SPEC): Pass -k to the assembler when 

gcc/cppmain.c

@@ -39,6 +39,7 @@ struct printer
 
 int main		PARAMS ((int, char **));
 static void general_init PARAMS ((const char *));
+static void do_preprocessing PARAMS ((int, char **));
 static void setup_callbacks PARAMS ((void));
 
 /* General output routines.  */
@@ -63,7 +64,6 @@ static void cb_def_pragma PARAMS ((cpp_reader *));
 const char *progname;		/* Needs to be global.  */
 static cpp_reader *pfile;	/* An opaque handle.  */
 static cpp_options *options;	/* Options of pfile.  */
-static cpp_callbacks *cb;	/* Callbacks of pfile.  */
 static struct printer print;
 
 int
@@ -71,39 +71,85 @@ main (argc, argv)
      int argc;
      char **argv;
 {
-  int argi = 1;  /* Next argument to handle.  */
-
   general_init (argv[0]);
-  /* Default language is GNU C89.  */
+
+  /* Contruct a reader with default language GNU C89.  */
   pfile = cpp_create_reader (CLK_GNUC89);
   options = cpp_get_options (pfile);
-  cb = cpp_get_callbacks (pfile);
+  
+  do_preprocessing (argc, argv);
+
+  /* Reader destructor.  */
+  cpp_cleanup (pfile);
+
+  if (cpp_errors (pfile))
+    return FATAL_EXIT_CODE;
+
+  return SUCCESS_EXIT_CODE;
+}
+
+/* Store the program name, and set the locale.  */
+static void
+general_init (const char *argv0)
+{
+  progname = argv0 + strlen (argv0);
+
+  while (progname != argv0 && ! IS_DIR_SEPARATOR (progname[-1]))
+    --progname;
+
+  xmalloc_set_program_name (progname);
+
+/* LC_CTYPE determines the character set used by the terminal so it
+   has to be set to output messages correctly.  */
+
+#ifdef HAVE_LC_MESSAGES
+  setlocale (LC_CTYPE, "");
+  setlocale (LC_MESSAGES, "");
+#else
+  setlocale (LC_ALL, "");
+#endif
+
+  (void) bindtextdomain (PACKAGE, localedir);
+  (void) textdomain (PACKAGE);
+}
+
+/* Handle switches, preprocess and output.  */
+static void
+do_preprocessing (argc, argv)
+     int argc;
+     char **argv;
+{
+  int argi = 1;  /* Next argument to handle.  */
 
   argi += cpp_handle_options (pfile, argc - argi , argv + argi);
-  if (argi < argc && ! CPP_FATAL_ERRORS (pfile))
+  if (CPP_FATAL_ERRORS (pfile))
+    return;
+
+  if (argi < argc)
     cpp_fatal (pfile, "Invalid option %s", argv[argi]);
+  else
     cpp_post_options (pfile);
+
   if (CPP_FATAL_ERRORS (pfile))
-    return (FATAL_EXIT_CODE);
+    return;
 
   /* If cpp_handle_options saw --help or --version on the command
      line, it will have set pfile->help_only to indicate this.  Exit
      successfully.  [The library does not exit itself, because
      e.g. cc1 needs to print its own --help message at this point.]  */
   if (options->help_only)
-    return (SUCCESS_EXIT_CODE);
+    return;
 
   /* Open the output now.  We must do so even if no_output is on,
      because there may be other output than from the actual
      preprocessing (e.g. from -dM).  */
   if (printer_init (pfile))
-    return (FATAL_EXIT_CODE);
+    return;
 
   setup_callbacks ();
 
-  if (! cpp_start_read (pfile, options->in_fname))
-    return (FATAL_EXIT_CODE);
-
+  if (cpp_start_read (pfile, options->in_fname))
+    {
       /* A successful cpp_start_read guarantees that we can call
 	 cpp_scan_buffer_nooutput or cpp_get_token next.  */
       if (options->no_output)
@@ -111,54 +157,27 @@ main (argc, argv)
       else
 	scan_buffer (pfile);
 
-  /* -dM command line option.  */
+      /* -dM command line option.  Should this be in cpp_finish?  */
       if (options->dump_macros == dump_only)
 	cpp_forall_identifiers (pfile, dump_macro, NULL);
 
       cpp_finish (pfile);
-  cpp_cleanup (pfile);
+    }
 
   /* Flush any pending output.  */
   if (print.printed)
     putc ('\n', print.outf);
+
   if (ferror (print.outf) || fclose (print.outf))
     cpp_notice_from_errno (pfile, options->out_fname);
-
-  if (cpp_errors (pfile))
-    return FATAL_EXIT_CODE;
-
-  return SUCCESS_EXIT_CODE;
-}
-
-/* Store the program name, and set the locale.  */
-static void
-general_init (const char *argv0)
-{
-  progname = argv0 + strlen (argv0);
-
-  while (progname != argv0 && ! IS_DIR_SEPARATOR (progname[-1]))
-    --progname;
-
-  xmalloc_set_program_name (progname);
-
-/* LC_CTYPE determines the character set used by the terminal so it has be set
-   to output messages correctly.  */
-
-#ifdef HAVE_LC_MESSAGES
-  setlocale (LC_CTYPE, "");
-  setlocale (LC_MESSAGES, "");
-#else
-  setlocale (LC_ALL, "");
-#endif
-
-  (void) bindtextdomain (PACKAGE, localedir);
-  (void) textdomain (PACKAGE);
 }
 
 /* Set up the callbacks as appropriate.  */
 static void
 setup_callbacks ()
 {
+  cpp_callbacks *cb = cpp_get_callbacks (pfile);
+
   if (! options->no_output)
     {
       cb->ident      = cb_ident;
@@ -264,10 +283,11 @@ printer_init (pfile)
       print.outf = fopen (options->out_fname, "w");
       if (! print.outf)
 	{
-	  cpp_notice_from_errno (pfile, options-> out_fname);
+	  cpp_notice_from_errno (pfile, options->out_fname);
 	  return 1;
 	}
     }
+
   return 0;
 }
 
@@ -324,7 +344,7 @@ print_line (special_flags)
 	   print.lineno, print.last_fname, special_flags, print.syshdr_flags);
 }
 
-/* Callbacks */
+/* Callbacks.  */
 
 static void
 cb_ident (pfile, str)