23.0.60; :enable (mark-active)

23.0.60; :enable (mark-active)

[Lennart Borgman (gmail)]

There are a couple of

   :enable (mark-active)

in lisp-mode.el. There is no function mark-active, should not the line 
above be

   :enable 'mark-active


In GNU Emacs 23.0.60.1 (i386-mingw-nt5.1.2600)
  of 2008-03-05
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (3.4) --no-opt --cflags 
-Ic:/g/include -fno-crossjumping'

Re: 23.0.60; :enable (mark-active)

[Bastien]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:

> There are a couple of
>
>   :enable (mark-active)
>
> in lisp-mode.el. There is no function mark-active, should not the line
> above be
>
>   :enable 'mark-active

It should be 

    :enable mark-active

I will fix this, thanks!

-- 
Bastien

Re: 23.0.60; :enable (mark-active)

[Lennart Borgman (gmail)]

Bastien wrote:
> "Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:
> 
>> There are a couple of
>>
>>   :enable (mark-active)
>>
>> in lisp-mode.el. There is no function mark-active, should not the line
>> above be
>>
>>   :enable 'mark-active
> 
> It should be 
> 
>     :enable mark-active
> 
> I will fix this, thanks!

Thanks, but I usually use 'mark-active. Which one is correct here?

Re: 23.0.60; :enable (mark-active)

[Andreas Schwab]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:

> Bastien wrote:
>> "Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:
>>
>>> There are a couple of
>>>
>>>   :enable (mark-active)
>>>
>>> in lisp-mode.el. There is no function mark-active, should not the line
>>> above be
>>>
>>>   :enable 'mark-active
>>
>> It should be 
>>
>>     :enable mark-active
>>
>> I will fix this, thanks!
>
> Thanks, but I usually use 'mark-active. Which one is correct here?

Depends on where it is used.  The use here is inside a quoted form,
where 'mark-active would be included as-is (and subsequently always
non-nil when evaluated).

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux Products GmbH, Maxfeldstraße 5, 90409 Nürnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: 23.0.60; :enable (mark-active)

[Bastien]

"Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:

>> It should be 
>>
>>     :enable mark-active
>>
>> I will fix this, thanks!
>
> Thanks, but I usually use 'mark-active. Which one is correct here?

It's 

  :enable mark-active

If you grep the code, the places where you see 

  :enable 'some-variable

are thos where the is part of

  (list ... :enable 'some-variable)

-- 
Bastien

Re: 23.0.60; :enable (mark-active)

[Lennart Borgman (gmail)]

Bastien wrote:
> "Lennart Borgman (gmail)" <lennart.borgman@gmail.com> writes:
> 
>>> It should be 
>>>
>>>     :enable mark-active
>>>
>>> I will fix this, thanks!
>> Thanks, but I usually use 'mark-active. Which one is correct here?
> 
> It's 
> 
>   :enable mark-active
> 
> If you grep the code, the places where you see 
> 
>   :enable 'some-variable
> 
> are thos where the is part of
> 
>   (list ... :enable 'some-variable)

Ah, yes, I did not notice it was in a quoted form.

RE: 23.0.60; :enable (mark-active)

[Drew Adams]

> > It should be 
> >     :enable mark-active
> > I will fix this, thanks!
> 
> Thanks, but I usually use 'mark-active. Which one is correct here?

'mark-active