I added this TODO about a year ago, because at that point I wasn't
completely sure if this particular check was needed or not. It still
looks like a no-op to me -- the only things that affect the "offset"
variable are that it's set to zero some dozen lines above the patch
region, and of course the two lines of context in the top of the patch
-- and so I'm now proposing to remove it.
-------------------------------------------------------------------
2007-03-23 Brooks Moses <[EMAIL PROTECTED]>
* error.c (show_locus): Remove always-false test.
-------------------------------------------------------------------
Tested with "make check-fortran" on i686-pc-linux-gnu. Ok for mainline?
- Brooks
Index: error.c
===================================================================
--- error.c (revision 123170)
+++ error.c (working copy)
@@ -233,12 +233,6 @@
if (cmax > terminal_width - 5)
offset = cmax - terminal_width + 5;
- /* TODO: Is there a good reason for the following apparently-redundant
- check, and the similar ones in the single-locus cases below? */
-
- if (offset < 0)
- offset = 0;
-
/* Show the line itself, taking care not to print more than what can
show up on the terminal. Tabs are converted to spaces, and
nonprintable characters are converted to a "\xNN" sequence. */
