On 01/09/2012 10:02 AM, Gary Funck wrote:
On 01/06/12 08:34:52, Tom Tromey wrote:
The patch is ok with either that change or with those 2 lines removed.

Applied as svn version 183003.

I get the build failure:

libcpp/macro.c:220:26: error: variable 'map' set but not used [-Werror=unused-but-set-variable]

Your patch removed:

- map = linemap_lookup (pfile->line_table, pfile->line_table->highest_line);
-           while (! MAIN_FILE_P (map))
-             map = INCLUDED_FROM (pfile->line_table, map);
-           name = ORDINARY_MAP_FILE_NAME (map);



I concur with the compiler, given that only the following remains:


_cpp_builtin_macro_text (cpp_reader *pfile, cpp_hashnode *node)
{
  const struct line_map *map;
...
    case BT_SPECLINE:
      map = LINEMAPS_LAST_ORDINARY_MAP (pfile->line_table);

which looks like dead code to me.

Tobias

Reply via email to