[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
FYI: Both: Cleanups
From: |
Akim Demaille |
Subject: |
FYI: Both: Cleanups |
Date: |
14 Nov 2001 15:21:43 +0100 |
User-agent: |
Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Artificial Intelligence) |
Index: ChangeLog
from Akim Demaille <address@hidden>
* src/lex.c, src/lex.h (token_buffer, unlexed_token_buffer):
Revert a previous patch: these are really const.
* src/conflicts.c (conflict_report): Additional useless pair of
braces to pacify GCC's warnings for `if () if () {} else {}'.
* src/lex.c (parse_percent_token): Replace equal_offset with
arg_offset.
arg is const.
Be sure to strdup `arg' when used, since there is no reason for
token_buffer not to change.
Index: src/conflicts.c
===================================================================
RCS file: /cvsroot/bison/bison/src/conflicts.c,v
retrieving revision 1.22.2.6
diff -u -u -r1.22.2.6 conflicts.c
--- src/conflicts.c 2001/11/12 09:19:48 1.22.2.6
+++ src/conflicts.c 2001/11/14 14:36:09
@@ -474,24 +474,26 @@
/* Report the total number of conflicts on STDERR. */
if (src_total || rrc_total)
- if (yacc_flag)
- {
- /* If invoked with `--yacc', use the output format specified by
- POSIX. */
- fprintf (stderr, _("conflicts: "));
- if (src_total > 0)
- fprintf (stderr, _(" %d shift/reduce"), src_total);
- if (src_total > 0 && rrc_total > 0)
- fprintf (stderr, ",");
- if (rrc_total > 0)
- fprintf (stderr, _(" %d reduce/reduce"), rrc_total);
- putc ('\n', stderr);
- }
- else
- {
- fprintf (stderr, _("%s contains "), infile);
- fputs (conflict_report (src_total, rrc_total), stderr);
- }
+ {
+ if (yacc_flag)
+ {
+ /* If invoked with `--yacc', use the output format specified by
+ POSIX. */
+ fprintf (stderr, _("conflicts: "));
+ if (src_total > 0)
+ fprintf (stderr, _(" %d shift/reduce"), src_total);
+ if (src_total > 0 && rrc_total > 0)
+ fprintf (stderr, ",");
+ if (rrc_total > 0)
+ fprintf (stderr, _(" %d reduce/reduce"), rrc_total);
+ putc ('\n', stderr);
+ }
+ else
+ {
+ fprintf (stderr, _("%s contains "), infile);
+ fputs (conflict_report (src_total, rrc_total), stderr);
+ }
+ }
if (expected_conflicts != -1
&& src_total != expected_conflicts)
Index: src/lex.c
===================================================================
RCS file: /cvsroot/bison/bison/src/lex.c,v
retrieving revision 1.33.2.11
diff -u -u -r1.33.2.11 lex.c
--- src/lex.c 2001/11/05 10:06:27 1.33.2.11
+++ src/lex.c 2001/11/14 14:36:09
@@ -29,15 +29,15 @@
/* Buffer for storing the current token. */
struct obstack token_obstack;
-char *token_buffer = NULL;
+const char *token_buffer = NULL;
-bucket *symval;
+bucket *symval = NULL;
int numval;
/* A token to be reread, see unlex and lex. */
static token_t unlexed = tok_undef;
static bucket *unlexed_symval = NULL;
-static char *unlexed_token_buffer = NULL;
+static const char *unlexed_token_buffer = NULL;
void
lex_init (void)
@@ -585,9 +585,9 @@
parse_percent_token (void)
{
struct percent_table_struct *tx = NULL;
- /* Where `=' was found in token_buffer. */
- size_t equal_offset = 0;
- char *arg = NULL;
+ const char *arg = NULL;
+ /* Where the ARG was found in token_buffer. */
+ size_t arg_offset = 0;
int c = getc (finput);
@@ -629,33 +629,34 @@
c = getc (finput);
}
+ /* %DIRECTIVE="ARG". Separate into
+ TOKEN_BUFFER = `%DIRECTIVE\0ARG\0'.
+ This is a bit hackish, but once we move to a Bison parser,
+ things will be cleaned up. */
if (c == '=')
{
- equal_offset = obstack_object_size (&token_obstack);
- obstack_1grow (&token_obstack, c);
+ /* End of the directive. We skip the `='. */
+ obstack_1grow (&token_obstack, '\0');
+ /* Fetch the ARG if present. */
c = getc (finput);
- if (c = '"')
+ if (c == '"')
{
- int code; /* ignored here */
-
- obstack_1grow (&token_obstack, '"');
- /* Read up to and including ". */
- while (literalchar (&token_obstack, &code, '"'))
- /* nothing */;
+ int code;
+ arg_offset = obstack_object_size (&token_obstack);
+ /* Read up to and including `"'. Do not append the closing
+ `"' in the output: it's not part of the ARG. */
+ while (literalchar (NULL, &code, '"'))
+ obstack_1grow (&token_obstack, code);
}
+ /* else: should be an error. */
}
else
ungetc (c, finput);
obstack_1grow (&token_obstack, '\0');
token_buffer = obstack_finish (&token_obstack);
- if (equal_offset)
- {
- /* %token_buffer="arg" */
- arg = token_buffer + equal_offset + 2;
- arg[strlen (arg) - 1] = '\0';
- token_buffer[equal_offset] = '\0';
- }
+ if (arg_offset)
+ arg = token_buffer + arg_offset;
/* table lookup % directive */
for (tx = percent_table; tx->name; tx++)
@@ -676,7 +677,7 @@
precedence. Side effect: if this %-option is used
several times, only the first is honored. Bah. */
if (!*((char **) (tx->set_flag)))
- *((char **) (tx->set_flag)) = arg;
+ *((char **) (tx->set_flag)) = xstrdup (arg);
}
else
fatal (_("`%s' requires an argument"), token_buffer);
Index: src/lex.h
===================================================================
RCS file: /cvsroot/bison/bison/src/lex.h,v
retrieving revision 1.13.2.5
diff -u -u -r1.13.2.5 lex.h
--- src/lex.h 2001/11/04 16:26:59 1.13.2.5
+++ src/lex.h 2001/11/14 14:36:09
@@ -55,7 +55,7 @@
tok_obsolete
} token_t;
-extern char *token_buffer;
+extern const char *token_buffer;
extern bucket *symval;
extern int numval;
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- FYI: Both: Cleanups,
Akim Demaille <=