Re: Bug in emacsbug.el

[Lennart Borgman <lennart.borgman.073@student.lu.se>]

Michaël Cadilhac wrote:
> Lennart Borgman <lennart.borgman.073@student.lu.se> writes:
> 
>> I think this test in `report-emacs-bug' tests the wrong mail address:
>>
>>       ;; Insert warnings for novice users.
>>       (when (string-match "@gnu\\.org^" reporting-address)
>> 	(insert "This bug report will be sent to the Free Software Foundation,\n")
>> 	(let ((pos (point)))
>> 	  (insert "not to your local site managers!")
>> 	  (put-text-property pos (point) 'face 'highlight)))
>>
>> Should it not test `user-mail-address' instead?
> 
> Don't think so.  This warning is here to warn the user that the email
> (s)he writes is GOING TO the FSF, not to the local sysadmin.
> 
> Why do you want to test user-mail-address? Maybe something's missing
> in the warning set of emacsbug in your opinion?


Oh, sorry. It is the regexp that is bad I think. It should probably be 
"@gnu\\.org$".

But I noted more strange things in `report-emacs-bug'. There are a lot 
of statements like

     (put-text-property pos (point) 'face 'highlight)

They does not seem to work. If the OBJECT parameter to put-text-property 
is a string it works, but it does not work with (current-buffer) as far 
as I can see.