https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68029
--- Comment #8 from Manuel López-Ibáñez <manu at gcc dot gnu.org> --- (In reply to Jiří Engelthaler from comment #7) > Created attachment 36638 [details] > diag_color.patch > > fdiagnostics_color_idx can by on first place so comparing as > 1 will miss > it. > There should be >= 1 Good catch! In principle your patch seems correct. I think you are only missing a testcase. This patch is small enough to not require a copyright assignment: https://gcc.gnu.org/wiki/GettingStarted#Basics:_Contributing_to_GCC_in_10_easy_steps