Misleading comment in simple.el?

Misleading comment in simple.el?

[Juanma Barranquero]

What does it mean this comment in simple.el?

  ;Turned off because it makes dbx bomb out.
  (setq blink-paren-function 'blink-matching-open)

`blink-matching-paren' defaults to t, so I don't undertand what it is
supposed to be "turned off". Perhaps the comment is a leftover?

             Juanma

Re: Misleading comment in simple.el?

[David Kastrup]

"Juanma Barranquero" <lekktu@gmail.com> writes:

> What does it mean this comment in simple.el?
>
>  ;Turned off because it makes dbx bomb out.
>  (setq blink-paren-function 'blink-matching-open)
>
> `blink-matching-paren' defaults to t, so I don't undertand what it is
> supposed to be "turned off". Perhaps the comment is a leftover?

C-x v g is pretty good for finding out things like that.

-- 
David Kastrup

Re: Misleading comment in simple.el?

[Juanma Barranquero]

On 3/30/07, David Kastrup <dak@gnu.org> wrote:

> C-x v g is pretty good for finding out things like that.

I had already tried it, and not, is not "pretty good" for finding that
kind of info. C-x v g allows me to know that the comment is there
since 1.1 (1999-12-21), and blink-matching-paren is also there since
1.1, and already set to t. It does not help me to know what was the
comment really refering to, unless I'm willing to check every single
commit since 1.1 on.

And no, I'm not asking anyone to research it for me; just if someone
remembers something that could shed a light over it.

             Juanma

Re: Misleading comment in simple.el?

[Richard Stallman]

      ;Turned off because it makes dbx bomb out.
      (setq blink-paren-function 'blink-matching-open)

    `blink-matching-paren' defaults to t, so I don't undertand what it is
    supposed to be "turned off". Perhaps the comment is a leftover?

I can't remember what it means, but it was there even in version 1.1
of the file.  So I guess we should just delete it.