From: Tim Janik Date: Tue, 30 Jun 1998 11:04:49 +0000 (+0000) Subject: take symbol_2_token into account. react on valid/invalid string pairs. X-Git-Tag: GLIB_1_1_0~20 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b7c94a718dc796adfb5515956073d061e7cfc4fe;p=platform%2Fupstream%2Fglib.git take symbol_2_token into account. react on valid/invalid string pairs. Tue Jun 30 11:58:25 1998 Tim Janik * gscanner.c (g_scanner_unexp_token): take symbol_2_token into account. react on valid/invalid string pairs. --- diff --git a/ChangeLog b/ChangeLog index e9d6dd7..20d7134 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-0 b/ChangeLog.pre-2-0 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-0 +++ b/ChangeLog.pre-2-0 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-10 +++ b/ChangeLog.pre-2-10 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-12 +++ b/ChangeLog.pre-2-12 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-2 b/ChangeLog.pre-2-2 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-2 +++ b/ChangeLog.pre-2-2 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-4 +++ b/ChangeLog.pre-2-4 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-6 +++ b/ChangeLog.pre-2-6 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8 index e9d6dd7..20d7134 100644 --- a/ChangeLog.pre-2-8 +++ b/ChangeLog.pre-2-8 @@ -1,3 +1,8 @@ +Tue Jun 30 11:58:25 1998 Tim Janik + + * gscanner.c (g_scanner_unexp_token): take symbol_2_token into + account. react on valid/invalid string pairs. + Sat Jun 27 21:55:46 PDT 1998 Manish Singh * glib.m4: ftp.glib.org -> ftp.gtk.org, since glib.org isn't diff --git a/glib/gscanner.c b/glib/gscanner.c index 41fcb4a..d401c91 100644 --- a/glib/gscanner.c +++ b/glib/gscanner.c @@ -679,12 +679,12 @@ g_scanner_unexp_token (GScanner *scanner, void (*msg_handler) (GScanner*, const gchar*, ...); g_return_if_fail (scanner != NULL); - + if (is_error) msg_handler = g_scanner_error; else msg_handler = g_scanner_warn; - + if (!identifier_spec) identifier_spec = "identifier"; if (!symbol_spec) @@ -698,12 +698,11 @@ g_scanner_unexp_token (GScanner *scanner, switch (scanner->token) { - case G_TOKEN_EOF: g_snprintf (token_string, token_string_len, "end of file"); break; - default: /* 1 ... 255 */ + default: if (scanner->token >= 1 && scanner->token <= 255) { if ((scanner->token >= ' ' && scanner->token <= '~') || @@ -712,9 +711,32 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (token_string, expected_string_len, "character `%c'", scanner->token); else g_snprintf (token_string, expected_string_len, "character `\\%o'", scanner->token); + break; } + else if (!scanner->config->symbol_2_token) + { + g_snprintf (token_string, token_string_len, "(unknown) token <%d>", scanner->token); + break; + } + /* fall through */ + case G_TOKEN_SYMBOL: + if (expected_token == G_TOKEN_SYMBOL || + (scanner->config->symbol_2_token && + expected_token > G_TOKEN_LAST)) + print_unexp = FALSE; + if (symbol_name) + g_snprintf (token_string, + token_string_len, + "%s%s `%s'", + print_unexp ? "" : "invalid ", + symbol_spec, + symbol_name); else - g_snprintf (token_string, token_string_len, "(unknown) token <%d>", scanner->token); + g_snprintf (token_string, + token_string_len, + "%s%s", + print_unexp ? "" : "invalid ", + symbol_spec); break; case G_TOKEN_ERROR: @@ -761,33 +783,15 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (token_string, token_string_len, "character `%c'", scanner->value.v_char); break; - case G_TOKEN_SYMBOL: - if (expected_token == G_TOKEN_SYMBOL) - print_unexp = FALSE; - if (symbol_name) - g_snprintf (token_string, - token_string_len, - "%s%s `%s'", - print_unexp ? "" : "invalid ", - symbol_spec, - symbol_name); - else - g_snprintf (token_string, - token_string_len, - "%s%s", - print_unexp ? "" : "invalid ", - symbol_spec); - break; - case G_TOKEN_IDENTIFIER: if (expected_token == G_TOKEN_IDENTIFIER) print_unexp = FALSE; g_snprintf (token_string, - token_string_len, - "%s%s `%s'", - print_unexp ? "" : "invalid ", - identifier_spec, - scanner->value.v_string); + token_string_len, + "%s%s `%s'", + print_unexp ? "" : "invalid ", + identifier_spec, + scanner->value.v_string); break; case G_TOKEN_BINARY: @@ -802,11 +806,14 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_STRING: + if (expected_token == G_TOKEN_STRING) + print_unexp = FALSE; g_snprintf (token_string, - token_string_len, - "%sstring constant \"%s\"", - scanner->value.v_string[0] == 0 ? "empty " : "", - scanner->value.v_string); + token_string_len, + "%s%sstring constant \"%s\"", + print_unexp ? "" : "invalid ", + scanner->value.v_string[0] == 0 ? "empty " : "", + scanner->value.v_string); token_string[token_string_len - 2] = '"'; token_string[token_string_len - 1] = 0; break; @@ -817,6 +824,12 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_NONE: + /* somehow the user's parsing code is screwed, there isn't much + * we can do about it. + * Note, a common case to trigger this is + * g_scanner_peek_next_token(); g_scanner_unexp_token(); + * without an intermediate g_scanner_get_next_token(). + */ g_assert_not_reached (); break; } @@ -824,7 +837,9 @@ g_scanner_unexp_token (GScanner *scanner, switch (expected_token) { - default: /* 1 ... 255 */ + gboolean need_valid; + + default: if (expected_token >= 1 && expected_token <= 255) { if ((expected_token >= ' ' && expected_token <= '~') || @@ -833,9 +848,24 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (expected_string, expected_string_len, "character `%c'", expected_token); else g_snprintf (expected_string, expected_string_len, "character `\\%o'", expected_token); + break; } - else - g_snprintf (expected_string, expected_string_len, "(unknown) token <%d>", expected_token); + else if (!scanner->config->symbol_2_token) + { + g_snprintf (expected_string, expected_string_len, "(unknown) token <%d>", expected_token); + break; + } + /* fall through */ + case G_TOKEN_SYMBOL: + need_valid = (scanner->token == G_TOKEN_SYMBOL || + (scanner->config->symbol_2_token && + scanner->token > G_TOKEN_LAST)); + g_snprintf (expected_string, + expected_string_len, + "%s%s", + need_valid ? "valid " : "", + symbol_spec); + /* FIXME: should we attempt to lookup the symbol_name for symbol_2_token? */ break; case G_TOKEN_INT: @@ -847,23 +877,18 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_STRING: - g_snprintf (expected_string, expected_string_len, "string constant"); - break; - - case G_TOKEN_SYMBOL: g_snprintf (expected_string, - expected_string_len, - "%s%s", - scanner->token == G_TOKEN_SYMBOL ? "valid " : "", - symbol_spec); + expected_string_len, + "%sstring constant", + scanner->token == G_TOKEN_STRING ? "valid " : ""); break; case G_TOKEN_IDENTIFIER: g_snprintf (expected_string, - expected_string_len, - "%s%s", - scanner->token == G_TOKEN_IDENTIFIER ? "valid " : "", - identifier_spec); + expected_string_len, + "%s%s", + scanner->token == G_TOKEN_IDENTIFIER ? "valid " : "", + identifier_spec); break; case G_TOKEN_NONE: diff --git a/gscanner.c b/gscanner.c index 41fcb4a..d401c91 100644 --- a/gscanner.c +++ b/gscanner.c @@ -679,12 +679,12 @@ g_scanner_unexp_token (GScanner *scanner, void (*msg_handler) (GScanner*, const gchar*, ...); g_return_if_fail (scanner != NULL); - + if (is_error) msg_handler = g_scanner_error; else msg_handler = g_scanner_warn; - + if (!identifier_spec) identifier_spec = "identifier"; if (!symbol_spec) @@ -698,12 +698,11 @@ g_scanner_unexp_token (GScanner *scanner, switch (scanner->token) { - case G_TOKEN_EOF: g_snprintf (token_string, token_string_len, "end of file"); break; - default: /* 1 ... 255 */ + default: if (scanner->token >= 1 && scanner->token <= 255) { if ((scanner->token >= ' ' && scanner->token <= '~') || @@ -712,9 +711,32 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (token_string, expected_string_len, "character `%c'", scanner->token); else g_snprintf (token_string, expected_string_len, "character `\\%o'", scanner->token); + break; } + else if (!scanner->config->symbol_2_token) + { + g_snprintf (token_string, token_string_len, "(unknown) token <%d>", scanner->token); + break; + } + /* fall through */ + case G_TOKEN_SYMBOL: + if (expected_token == G_TOKEN_SYMBOL || + (scanner->config->symbol_2_token && + expected_token > G_TOKEN_LAST)) + print_unexp = FALSE; + if (symbol_name) + g_snprintf (token_string, + token_string_len, + "%s%s `%s'", + print_unexp ? "" : "invalid ", + symbol_spec, + symbol_name); else - g_snprintf (token_string, token_string_len, "(unknown) token <%d>", scanner->token); + g_snprintf (token_string, + token_string_len, + "%s%s", + print_unexp ? "" : "invalid ", + symbol_spec); break; case G_TOKEN_ERROR: @@ -761,33 +783,15 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (token_string, token_string_len, "character `%c'", scanner->value.v_char); break; - case G_TOKEN_SYMBOL: - if (expected_token == G_TOKEN_SYMBOL) - print_unexp = FALSE; - if (symbol_name) - g_snprintf (token_string, - token_string_len, - "%s%s `%s'", - print_unexp ? "" : "invalid ", - symbol_spec, - symbol_name); - else - g_snprintf (token_string, - token_string_len, - "%s%s", - print_unexp ? "" : "invalid ", - symbol_spec); - break; - case G_TOKEN_IDENTIFIER: if (expected_token == G_TOKEN_IDENTIFIER) print_unexp = FALSE; g_snprintf (token_string, - token_string_len, - "%s%s `%s'", - print_unexp ? "" : "invalid ", - identifier_spec, - scanner->value.v_string); + token_string_len, + "%s%s `%s'", + print_unexp ? "" : "invalid ", + identifier_spec, + scanner->value.v_string); break; case G_TOKEN_BINARY: @@ -802,11 +806,14 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_STRING: + if (expected_token == G_TOKEN_STRING) + print_unexp = FALSE; g_snprintf (token_string, - token_string_len, - "%sstring constant \"%s\"", - scanner->value.v_string[0] == 0 ? "empty " : "", - scanner->value.v_string); + token_string_len, + "%s%sstring constant \"%s\"", + print_unexp ? "" : "invalid ", + scanner->value.v_string[0] == 0 ? "empty " : "", + scanner->value.v_string); token_string[token_string_len - 2] = '"'; token_string[token_string_len - 1] = 0; break; @@ -817,6 +824,12 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_NONE: + /* somehow the user's parsing code is screwed, there isn't much + * we can do about it. + * Note, a common case to trigger this is + * g_scanner_peek_next_token(); g_scanner_unexp_token(); + * without an intermediate g_scanner_get_next_token(). + */ g_assert_not_reached (); break; } @@ -824,7 +837,9 @@ g_scanner_unexp_token (GScanner *scanner, switch (expected_token) { - default: /* 1 ... 255 */ + gboolean need_valid; + + default: if (expected_token >= 1 && expected_token <= 255) { if ((expected_token >= ' ' && expected_token <= '~') || @@ -833,9 +848,24 @@ g_scanner_unexp_token (GScanner *scanner, g_snprintf (expected_string, expected_string_len, "character `%c'", expected_token); else g_snprintf (expected_string, expected_string_len, "character `\\%o'", expected_token); + break; } - else - g_snprintf (expected_string, expected_string_len, "(unknown) token <%d>", expected_token); + else if (!scanner->config->symbol_2_token) + { + g_snprintf (expected_string, expected_string_len, "(unknown) token <%d>", expected_token); + break; + } + /* fall through */ + case G_TOKEN_SYMBOL: + need_valid = (scanner->token == G_TOKEN_SYMBOL || + (scanner->config->symbol_2_token && + scanner->token > G_TOKEN_LAST)); + g_snprintf (expected_string, + expected_string_len, + "%s%s", + need_valid ? "valid " : "", + symbol_spec); + /* FIXME: should we attempt to lookup the symbol_name for symbol_2_token? */ break; case G_TOKEN_INT: @@ -847,23 +877,18 @@ g_scanner_unexp_token (GScanner *scanner, break; case G_TOKEN_STRING: - g_snprintf (expected_string, expected_string_len, "string constant"); - break; - - case G_TOKEN_SYMBOL: g_snprintf (expected_string, - expected_string_len, - "%s%s", - scanner->token == G_TOKEN_SYMBOL ? "valid " : "", - symbol_spec); + expected_string_len, + "%sstring constant", + scanner->token == G_TOKEN_STRING ? "valid " : ""); break; case G_TOKEN_IDENTIFIER: g_snprintf (expected_string, - expected_string_len, - "%s%s", - scanner->token == G_TOKEN_IDENTIFIER ? "valid " : "", - identifier_spec); + expected_string_len, + "%s%s", + scanner->token == G_TOKEN_IDENTIFIER ? "valid " : "", + identifier_spec); break; case G_TOKEN_NONE: