X-Git-Url: https://git.saurik.com/bison.git/blobdiff_plain/82129ef5d5932d207d100c731de6de4aeb5c9a15..ae199bf10fc7f201c07593f4e4f85ec7f12d5764:/src/files.c diff --git a/src/files.c b/src/files.c index e92771ff..f2092b27 100644 --- a/src/files.c +++ b/src/files.c @@ -1,6 +1,6 @@ /* Open and close files for Bison. - Copyright (C) 1984, 1986, 1989, 1992, 2000, 2001, 2002 + Copyright (C) 1984, 1986, 1989, 1992, 2000, 2001, 2002, 2003, 2004, 2005 Free Software Foundation, Inc. This file is part of Bison, the GNU Compiler Compiler. @@ -17,19 +17,22 @@ You should have received a copy of the GNU General Public License along with Bison; see the file COPYING. If not, write to the Free - Software Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA - 02111-1307, USA. */ + Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA + 02110-1301, USA. */ #include "system.h" #include #include +#include +#include #include "complain.h" #include "files.h" #include "getargs.h" #include "gram.h" +#include "stdio-safer.h" /* From basename.c. Almost a lie, as it returns a char *. */ const char *base_name (char const *name); @@ -63,9 +66,9 @@ static char *full_base_name = NULL; char *short_base_name = NULL; /* C source file extension (the parser source). */ -const char *src_extension = NULL; +static char const *src_extension = NULL; /* Header file extension (if option ``-d'' is specified). */ -const char *header_extension = NULL; +static char const *header_extension = NULL; /*-----------------------------------------------------------------. | Return a newly allocated string composed of the concatenation of | @@ -93,7 +96,7 @@ xfopen (const char *name, const char *mode) { FILE *ptr; - ptr = fopen (name, mode); + ptr = fopen_safer (name, mode); if (!ptr) error (EXIT_FAILURE, get_errno (), _("cannot open file `%s'"), name); @@ -118,9 +121,9 @@ xfclose (FILE *ptr) } -/*----------------------------------------------------------------. -| Compute BASE_NAME, SHORT_BASE_NAME and output files extensions. | -`----------------------------------------------------------------*/ +/*---------------------------------------------------------------------. +| Compute FULL_BASE_NAME, SHORT_BASE_NAME and output files extensions. | +`---------------------------------------------------------------------*/ /* Replace all characters FROM by TO in the string IN. and returns a new allocated string. */ @@ -139,7 +142,7 @@ tr (const char *in, char from, char to) return (temp); } -/* Computes extensions from the grammar file extension. */ +/* Compute extensions from the grammar file extension. */ static void compute_exts_from_gf (const char *ext) { @@ -149,7 +152,7 @@ compute_exts_from_gf (const char *ext) header_extension = tr (header_extension, 'Y', 'H'); } -/* Computes extensions from the given c source file extension. */ +/* Compute extensions from the given c source file extension. */ static void compute_exts_from_src (const char *ext) { @@ -162,8 +165,8 @@ compute_exts_from_src (const char *ext) } -/* Decompose FILENAME in four parts: *BASE, *TAB, and *EXT, the fourth - part, (the directory) is ranging from FILENAME to the char before +/* Decompose FILE_NAME in four parts: *BASE, *TAB, and *EXT, the fourth + part, (the directory) is ranging from FILE_NAME to the char before *BASE, so we don't need an additional parameter. *EXT points to the last period in the basename, or NULL if none. @@ -172,7 +175,7 @@ compute_exts_from_src (const char *ext) `.tab' or `_tab' if present right before *EXT, or is NULL. *TAB cannot be equal to *BASE. - None are allocated, they are simply pointers to parts of FILENAME. + None are allocated, they are simply pointers to parts of FILE_NAME. Examples: '/tmp/foo.tab.c' -> *BASE = 'foo.tab.c', *TAB = '.tab.c', *EXT = @@ -191,10 +194,10 @@ compute_exts_from_src (const char *ext) 'foo' -> *BASE = 'foo', *TAB = NULL, *EXT = NULL. */ static void -filename_split (const char *filename, - const char **base, const char **tab, const char **ext) +file_name_split (const char *file_name, + const char **base, const char **tab, const char **ext) { - *base = base_name (filename); + *base = base_name (file_name); /* Look for the extension, i.e., look for the last dot. */ *ext = strrchr (*base, '.'); @@ -214,24 +217,22 @@ filename_split (const char *filename, } -/* FIXME: Should use xstrndup. */ - static void compute_base_names (void) { const char *base, *tab, *ext; /* If --output=foo.c was specified (SPEC_OUTFILE == foo.c), - BASE_NAME and SHORT_BASE_NAME are `foo'. + FULL_BASE_NAME and SHORT_BASE_NAME are `foo'. - If --output=foo.tab.c was specified, BASE_NAME is `foo.tab' and - SHORT_BASE_NAME is `foo'. + If --output=foo.tab.c was specified, FULL_BASE_NAME is `foo.tab' + and SHORT_BASE_NAME is `foo'. The precise -o name will be used for FTABLE. For other output files, remove the ".c" or ".tab.c" suffix. */ if (spec_outfile) { - filename_split (spec_outfile, &base, &tab, &ext); + file_name_split (spec_outfile, &base, &tab, &ext); /* The full base name goes up the EXT, excluding it. */ full_base_name = @@ -269,7 +270,7 @@ compute_base_names (void) { /* Otherwise, the short base name is computed from the input grammar: `foo/bar.yy' => `bar'. */ - filename_split (grammar_file, &base, &tab, &ext); + file_name_split (grammar_file, &base, &tab, &ext); short_base_name = xstrndup (base, (strlen (base) - (ext ? strlen (ext) : 0))); @@ -279,20 +280,25 @@ compute_base_names (void) + strlen (TAB_EXT) + 1); stpcpy (stpcpy (full_base_name, short_base_name), TAB_EXT); - /* Computes the extensions from the grammar file name. */ - filename_split (grammar_file, &base, &tab, &ext); + /* Compute the extensions from the grammar file name. */ + file_name_split (grammar_file, &base, &tab, &ext); if (ext && !yacc_flag) compute_exts_from_gf (ext); } } -/*-------------------------------------------------------. -| Close the open files, compute the output files names. | -`-------------------------------------------------------*/ + +/* Compute the output file names. Warn if we detect conflicting + outputs to the same file. */ void compute_output_file_names (void) { + char const *name[4]; + int i; + int j; + int names = 0; + compute_base_names (); /* If not yet done. */ @@ -301,16 +307,31 @@ compute_output_file_names (void) if (!header_extension) header_extension = ".h"; - parser_file_name = + name[names++] = parser_file_name = spec_outfile ? spec_outfile : concat2 (full_base_name, src_extension); - /* It the defines filename if not given, we create it. */ - if (!spec_defines_file) - spec_defines_file = concat2 (full_base_name, header_extension); + if (defines_flag) + { + if (! spec_defines_file) + spec_defines_file = concat2 (full_base_name, header_extension); + name[names++] = spec_defines_file; + } + + if (graph_flag) + { + if (! spec_graph_file) + spec_graph_file = concat2 (short_base_name, ".vcg"); + name[names++] = spec_graph_file; + } - /* It the graph filename if not given, we create it. */ - if (!spec_graph_file) - spec_graph_file = concat2 (short_base_name, ".vcg"); + if (report_flag) + { + spec_verbose_file = concat2 (short_base_name, OUTPUT_EXT); + name[names++] = spec_verbose_file; + } - spec_verbose_file = concat2 (short_base_name, OUTPUT_EXT); + for (j = 0; j < names; j++) + for (i = 0; i < j; i++) + if (strcmp (name[i], name[j]) == 0) + warn (_("conflicting outputs to file %s"), quote (name[i])); }