Re: BUG in nnml.el

Re: BUG in nnml.el

[Katsumi Yamaoka]

>>>>> In article <w8sspe4awms.fsf@hler.ifi.uio.no>
>>>>>   gnus-bug@ifi.uio.no (The Gnus Bugfixing Girls + Boys) writes:

> Katsumi Yamaoka <yamaoka@ga.sony.co.jp> writes:
>> nnml deletes unnecessary articles.
>> This is the patch for sgnus-0.73.
>>
>> --- nnml.el.orig	Tue Apr 16 14:56:03 1996
>> +++ nnml.el	Tue Apr 16 14:56:03 1996
>> @@ -306,7 +306,8 @@
>> (progn
>> (condition-case ()
>> (funcall nnmail-delete-file-function
>> -		    (concat nnml-current-directory
>> +		    (concat nnml-directory
>> +			    (substring gnus-newsgroup-name 5) "/"
>> (int-to-string article)))
>> (file-error nil))
>> (nnml-nov-delete-article group article)

> I'm sorry, but what bug does this fix?  `nnml-current-directory'
> should be the same as
> 		    (concat nnml-directory
> 			    (substring gnus-newsgroup-name 5) "/")

 This `nnml-current-directory' belongs to the group which the article
is moved to, but it should be current group.
 Then, think about a example when I try to move article
`nnml:inbox 123' to `nnml:ding'. Nnml copies `nnml:inbox 123' to
`nnml:ding 456', for example, and it deletes `nnml:ding 123' instead
of `nnml:inbox 123'.
-- 
Katsumi Yamaoka <yamaoka@ga.sony.co.jp>
MD Dep., Personal A&V Products Company, Sony Corporation, Tokyo, Japan.