diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 669297e125c..b995be20fa5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2001-08-07 Neil Booth + + * cppinit.c (cpp_finish): Pop the final buffer without comment. + * cpplex.c (_cpp_lex_token): Don't pop the final buffer; and + take care to avoid multiple no-newline at EOF warnings in that + case. + Tue Aug 7 22:18:06 CEST 2001 Jan Hubicka * calls.c (expand_call): Do not emit INSN_SETJMP note. diff --git a/gcc/cppinit.c b/gcc/cppinit.c index f1b6b24f0aa..5a3873464fc 100644 --- a/gcc/cppinit.c +++ b/gcc/cppinit.c @@ -1010,12 +1010,13 @@ void cpp_finish (pfile) cpp_reader *pfile; { - if (CPP_BUFFER (pfile)) - { - cpp_ice (pfile, "buffers still stacked in cpp_finish"); - while (CPP_BUFFER (pfile)) - _cpp_pop_buffer (pfile); - } + /* cpplex.c leaves the final buffer on the stack. This it so that + it returns an unending stream of CPP_EOFs to the client. If we + popped the buffer, we'd derefence a NULL buffer pointer and + segfault. It's nice to allow the client to do worry-free excess + cpp_get_token calls. */ + while (pfile->buffer) + _cpp_pop_buffer (pfile); /* Don't write the deps file if preprocessing has failed. */ if (CPP_OPTION (pfile, print_deps) && pfile->errors == 0) diff --git a/gcc/cpplex.c b/gcc/cpplex.c index f9c4bb9ab0c..5e0aa5a4f85 100644 --- a/gcc/cpplex.c +++ b/gcc/cpplex.c @@ -897,21 +897,26 @@ _cpp_lex_token (pfile, result) pfile->line--; else if (! pfile->state.parsing_args) { - unsigned char ret = pfile->buffer->return_at_eof; - /* Non-empty files should end in a newline. Don't warn for command line and _Pragma buffers. */ if (pfile->lexer_pos.col != 0) { - /* Account for the missing \n. */ + /* Account for the missing \n, prevent multiple warnings. */ pfile->line++; + pfile->lexer_pos.col = 0; if (!buffer->from_stage3) cpp_pedwarn (pfile, "no newline at end of file"); } - _cpp_pop_buffer (pfile); - if (pfile->buffer && !ret) - goto next_token; + /* Don't pop the last file. */ + if (buffer->prev) + { + unsigned char stop = buffer->return_at_eof; + + _cpp_pop_buffer (pfile); + if (!stop) + goto next_token; + } } result->type = CPP_EOF; return;