variable selective_display in search.c?

[Pavel Janík]

Do we really need it? It is not used, but has a two-line comment before it:

Index: search.c
===================================================================
RCS file: /cvsroot/emacs/emacs/src/search.c,v
retrieving revision 1.158
diff -u -r1.158 search.c
--- search.c    2001/10/21 12:05:35     1.158
+++ search.c    2001/10/24 06:39:12
@@ -752,11 +752,6 @@
 
   int old_immediate_quit = immediate_quit;
 
-  /* If we are not in selective display mode,
-     check only for newlines.  */
-  int selective_display = (!NILP (current_buffer->selective_display)
-                          && !INTEGERP (current_buffer->selective_display));
-
   /* The code that follows is like scan_buffer
      but checks for either newline or carriage return.  */

It was introduced by RMS on 1998/01/01 02:52:20 with this ChangeLog entry
(revision 1.98):

(scan_newline): New function.

This variable is not used now and was never used. Any objection to remove
it together with this comment (the same code with exactly the same comment
is also in xdisp.c and yes, we can find it in the CVS in the future if we
want it).
-- 
Pavel Janík

Hey, if you've mlock'ed more than your available memory, there's nothing
the VM layer can do. Except maybe a nice printk("Kiss your *ss goodbye");
                  -- Linus Torvalds in linux-kernel