Re: unused function in src/tables.c

[Paul Eggert]

Roland McGrath <address@hidden> writes:

> In bison-2.0 (and current CVS), this function is unused.
> GCC doesn't warn because it's an inline, but some other compilers might.

Thanks.  I installed this slightly different patch, since I think the
function's text is useful documentation.

2005-03-15  Paul Eggert  <address@hidden>

        * src/tables.c (state_number_to_vector_number): Put it inside an
        "#if 0", since it's not currently used.  Problem reported by
        Roland McGrath.

--- tables.c    17 Dec 2004 20:29:19 -0000      1.25
+++ tables.c    15 Mar 2005 23:53:27 -0000      1.26
@@ -1,7 +1,7 @@
 /* Output the generated parsing program for Bison.
 
-   Copyright (C) 1984, 1986, 1989, 1992, 2000, 2001, 2002, 2003, 2004
-   Free Software Foundation, Inc.
+   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.
 
@@ -44,11 +44,13 @@
    state_number and symbol_number.  */
 typedef int vector_number;
 
+#if 0 /* Not currently used.  */
 static inline vector_number
 state_number_to_vector_number (state_number s)
 {
   return s;
 }
+#endif
 
 static inline vector_number
 symbol_number_to_vector_number (symbol_number sym)