[Bug c/89549] [8/9/10 Regression] -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers

2020-03-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|8.4 |8.5

--- Comment #18 from Jakub Jelinek  ---
GCC 8.4.0 has been released, adjusting target milestone.

[Bug c/89549] [8/9/10 Regression] -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers

2019-11-25 Thread david.bolvansky at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549

--- Comment #17 from Dávid Bolvanský  ---
Check few lines above

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549#c8

[Bug c/89549] [8/9/10 Regression] -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers

2019-11-25 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549

--- Comment #16 from Jakub Jelinek  ---
Of course there is, -Wno-misleading-indentation or use line breaks from time to
time.  With >= 4KB long lines, the user clearly doesn't care about indentation,
so the warning doesn't make sense.

[Bug c/89549] [8/9/10 Regression] -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers

2019-11-25 Thread david.bolvansky at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549

--- Comment #15 from Dávid Bolvanský  ---
But there is no way to silence this "note".

[Bug c/89549] [8/9/10 Regression] -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers

2019-11-25 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89549

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #14 from Jakub Jelinek  ---
I don't see a bug here, it behaves as expected, once you have too long line
somewhere or too many lines, column tracking is disabled and column tracking is
essential for this warning:
  /* PR c++/68819: if the column number is zero, we presumably
 had a location_t > LINE_MAP_MAX_LOCATION_WITH_COLS, and so
 we have no column information.
 Act as if no conversion was possible, triggering the
 error-handling path in the caller.  */
  if (!exploc.column)
{
  static bool issued_note = false;
  if (!issued_note)
{
  /* Notify the user the first time this happens.  */
  issued_note = true;
  inform (loc,
  "%<-Wmisleading-indentation%> is disabled from this point"
  " onwards, since column-tracking was disabled due to"
  " the size of the code/headers");
}
  return false;
}
This isn't a note that is related to some particular warning, just a message
telling that there won't be any further warnings emitted due to this later.