public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug preprocessor/91733] No longer treat carriage return as an end-of-line [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> @ 2022-07-08 1:58 ` lhyatt at gcc dot gnu.org 2022-07-08 14:04 ` cvs-commit at gcc dot gnu.org ` (3 subsequent siblings) 4 siblings, 0 replies; 5+ messages in thread From: lhyatt at gcc dot gnu.org @ 2022-07-08 1:58 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91733 Lewis Hyatt <lhyatt at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lhyatt at gcc dot gnu.org --- Comment #4 from Lewis Hyatt <lhyatt at gcc dot gnu.org> --- Patch to fix the diagnostics issue submitted for review here: https://gcc.gnu.org/pipermail/gcc-patches/2022-July/598047.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug preprocessor/91733] No longer treat carriage return as an end-of-line [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> 2022-07-08 1:58 ` [Bug preprocessor/91733] No longer treat carriage return as an end-of-line lhyatt at gcc dot gnu.org @ 2022-07-08 14:04 ` cvs-commit at gcc dot gnu.org 2022-07-08 14:06 ` lhyatt at gcc dot gnu.org ` (2 subsequent siblings) 4 siblings, 0 replies; 5+ messages in thread From: cvs-commit at gcc dot gnu.org @ 2022-07-08 14:04 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91733 --- Comment #5 from CVS Commits <cvs-commit at gcc dot gnu.org> --- The master branch has been updated by Lewis Hyatt <lhyatt@gcc.gnu.org>: https://gcc.gnu.org/g:2bd15617e73acf76207127e5580cd10b1fab91a5 commit r13-1577-g2bd15617e73acf76207127e5580cd10b1fab91a5 Author: Lewis Hyatt <lhyatt@gmail.com> Date: Thu Jul 7 13:59:27 2022 -0400 diagnostics: Make line-ending logic consistent with libcpp [PR91733] libcpp recognizes a lone \r as a valid line ending, so the infrastructure for retrieving source lines to be output in diagnostics needs to do the same. This patch fixes file_cache_slot::get_next_line() accordingly so that diagnostics display the correct part of the source when \r line endings are in use. gcc/ChangeLog: PR preprocessor/91733 * input.cc (find_end_of_line): New helper function. (file_cache_slot::get_next_line): Recognize \r as a line ending. * diagnostic-show-locus.cc (test_escaping_bytes_1): Adapt selftest since \r will now be interpreted as a line-ending. gcc/testsuite/ChangeLog: PR preprocessor/91733 * c-c++-common/pr91733.c: New test. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug preprocessor/91733] No longer treat carriage return as an end-of-line [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> 2022-07-08 1:58 ` [Bug preprocessor/91733] No longer treat carriage return as an end-of-line lhyatt at gcc dot gnu.org 2022-07-08 14:04 ` cvs-commit at gcc dot gnu.org @ 2022-07-08 14:06 ` lhyatt at gcc dot gnu.org 2022-07-08 14:27 ` akim.demaille at gmail dot com 2022-07-10 15:25 ` dmalcolm at gcc dot gnu.org 4 siblings, 0 replies; 5+ messages in thread From: lhyatt at gcc dot gnu.org @ 2022-07-08 14:06 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91733 --- Comment #6 from Lewis Hyatt <lhyatt at gcc dot gnu.org> --- The diagnostics issue is fixed now. Do we want to mark this resolved, or keep it open given the other question regarding support for \r line endings in general? ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug preprocessor/91733] No longer treat carriage return as an end-of-line [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> ` (2 preceding siblings ...) 2022-07-08 14:06 ` lhyatt at gcc dot gnu.org @ 2022-07-08 14:27 ` akim.demaille at gmail dot com 2022-07-10 15:25 ` dmalcolm at gcc dot gnu.org 4 siblings, 0 replies; 5+ messages in thread From: akim.demaille at gmail dot com @ 2022-07-08 14:27 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91733 --- Comment #7 from Akim Demaille <akim.demaille at gmail dot com> --- Personally the bug I reported was the one you fixed. I merely suggested to drop \r, but I did asked for that. So AFAIC, you may close this issue. Thanks a lot for the fix! ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug preprocessor/91733] No longer treat carriage return as an end-of-line [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> ` (3 preceding siblings ...) 2022-07-08 14:27 ` akim.demaille at gmail dot com @ 2022-07-10 15:25 ` dmalcolm at gcc dot gnu.org 4 siblings, 0 replies; 5+ messages in thread From: dmalcolm at gcc dot gnu.org @ 2022-07-10 15:25 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91733 David Malcolm <dmalcolm at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dmalcolm at gcc dot gnu.org Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #8 from David Malcolm <dmalcolm at gcc dot gnu.org> --- Thanks; marking this one as RESOLVED. ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-07-10 15:25 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <bug-91733-4@http.gcc.gnu.org/bugzilla/> 2022-07-08 1:58 ` [Bug preprocessor/91733] No longer treat carriage return as an end-of-line lhyatt at gcc dot gnu.org 2022-07-08 14:04 ` cvs-commit at gcc dot gnu.org 2022-07-08 14:06 ` lhyatt at gcc dot gnu.org 2022-07-08 14:27 ` akim.demaille at gmail dot com 2022-07-10 15:25 ` dmalcolm at gcc dot gnu.org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).