Gnus development mailing list
 help / color / mirror / Atom feed
* Re: [gnus git]  branch master updated: =1= message: insert mail-header-separator read-only and intangible
       [not found] <E1Pl14t-0002by-00@quimby.gnus.org>
@ 2011-02-04  0:14 ` Katsumi Yamaoka
  2011-02-04  8:47   ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Katsumi Yamaoka @ 2011-02-04  0:14 UTC (permalink / raw)
  To: ding

Julien Danjou <julien@danjou.info> wrote:
> +	(message-setup-1): Insert mail-header-separator with read-only and
> +	intangible properties set.
[...]
> +  (let ((message-forbidden-properties nil))
> +    (insert (propertize (concat mail-header-separator "\n")
> +                        'read-only t 'rear-nonsticky t 'intangible t)))

Why does it need to let-bind?  Emacs 22 and 23 say:

In message-setup-1:
message.el:6447:10:Warning: attempt to let-bind constant
    `message-forbidden-properties'



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gnus git]  branch master updated: =1= message: insert mail-header-separator read-only and intangible
  2011-02-04  0:14 ` [gnus git] branch master updated: =1= message: insert mail-header-separator read-only and intangible Katsumi Yamaoka
@ 2011-02-04  8:47   ` Lars Ingebrigtsen
  2011-02-04 10:14     ` Julien Danjou
  0 siblings, 1 reply; 4+ messages in thread
From: Lars Ingebrigtsen @ 2011-02-04  8:47 UTC (permalink / raw)
  To: ding

Katsumi Yamaoka <yamaoka@jpl.org> writes:

>> +  (let ((message-forbidden-properties nil))
>> +    (insert (propertize (concat mail-header-separator "\n")
>> +                        'read-only t 'rear-nonsticky t 'intangible t)))
>
> Why does it need to let-bind?

Yes, that seems odd.  Binding that const there won't have any effect,
will it?

-- 
(domestic pets only, the antidote for overdose, milk.)
  larsi@gnus.org * Lars Magne Ingebrigtsen




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gnus git]  branch master updated: =1= message: insert mail-header-separator read-only and intangible
  2011-02-04  8:47   ` Lars Ingebrigtsen
@ 2011-02-04 10:14     ` Julien Danjou
  2011-02-04 10:15       ` Lars Ingebrigtsen
  0 siblings, 1 reply; 4+ messages in thread
From: Julien Danjou @ 2011-02-04 10:14 UTC (permalink / raw)
  To: ding

[-- Attachment #1: Type: text/plain, Size: 331 bytes --]

On Fri, Feb 04 2011, Lars Ingebrigtsen wrote:

>> Why does it need to let-bind?

Because otherwise the read-only will be removed.

> Yes, that seems odd.  Binding that const there won't have any effect,
> will it?

Well it does have an effect on Emacs 24 at least. ;)

-- 
Julien Danjou
❱ http://julien.danjou.info

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [gnus git]  branch master updated: =1= message: insert mail-header-separator read-only and intangible
  2011-02-04 10:14     ` Julien Danjou
@ 2011-02-04 10:15       ` Lars Ingebrigtsen
  0 siblings, 0 replies; 4+ messages in thread
From: Lars Ingebrigtsen @ 2011-02-04 10:15 UTC (permalink / raw)
  To: ding

Julien Danjou <julien@danjou.info> writes:

>> Yes, that seems odd.  Binding that const there won't have any effect,
>> will it?
>
> Well it does have an effect on Emacs 24 at least. ;)

Yeah, I discovered the after-change-hook thing...  sneaky...

-- 
(domestic pets only, the antidote for overdose, milk.)
  larsi@gnus.org * Lars Magne Ingebrigtsen




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-02-04 10:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E1Pl14t-0002by-00@quimby.gnus.org>
2011-02-04  0:14 ` [gnus git] branch master updated: =1= message: insert mail-header-separator read-only and intangible Katsumi Yamaoka
2011-02-04  8:47   ` Lars Ingebrigtsen
2011-02-04 10:14     ` Julien Danjou
2011-02-04 10:15       ` Lars Ingebrigtsen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).