Message ID | 3d84c0e9-6684-e74b-08e5-afc4b57acee4@126.com |
---|---|
State | New |
Headers | show |
Series | Two fixes for pretty-print.c for mingw-w64 | expand |
在 2018/8/14 13:54, Liu Hao 写道: > The two patches attached have addressed two issues in the ANSI escape > sequence translator I sent before. Please review, and consider > backporting these to gcc-8-branch. > > > > And here are SVN changelogs for these two patches, in this order: * gcc/pretty-print.c (mingw_ansi_fputs): Do not call _close() on the handle returned by _get_osf_handle(). * gcc/pretty-print.c (eat_esc_sequence): Swap the foreground and background colors if the COMMON_LVB_REVERSE_VIDEO flag is set, and clear it thereafter, as it only works for DBCS.
On 08/14/2018 05:55 AM, Liu Hao wrote: > 在 2018/8/14 13:54, Liu Hao 写道: >> The two patches attached have addressed two issues in the ANSI escape >> sequence translator I sent before. Please review, and consider >> backporting these to gcc-8-branch. >> >> >> >> > > And here are SVN changelogs for these two patches, in this order: > > > > * gcc/pretty-print.c (mingw_ansi_fputs): Do not call _close() on the handle > returned by _get_osf_handle(). > > > > * gcc/pretty-print.c (eat_esc_sequence): Swap the foreground and background > colors if the COMMON_LVB_REVERSE_VIDEO flag is set, and clear it > thereafter, > as it only works for DBCS. > > > Patch and log looks OK.
diff --git a/gcc/pretty-print.c b/gcc/pretty-print.c index 736af8f7735..31eb8893f2a 100644 --- a/gcc/pretty-print.c +++ b/gcc/pretty-print.c @@ -684,7 +684,6 @@ mingw_ansi_fputs (const char *str, FILE *fp) /* If it is not a console, write everything as-is. */ write_all (h, read, strlen (read)); - _close ((intptr_t) h); return 1; }