🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: gnu.emacs.vm.bug
Article #297

Re: Suggested fix to vm-pipe-message-part

#297
From: Uday S Reddy
Date: Fri, 08 Jan 2010 11:02
56 lines
4644 bytes
blueman wrote:

>> Perhaps it does.  But I already added a fix a day or two ago, tested
>> it and committed to the repository on launchpad.  Unfortunately, this
>> to-and-fro hasn't contributed all that much to the problem, after the
>> initial discussion.  There are some 20 odd messages in this thread.  I
>> don't think this mode of communication has been very effective.
> 
> Personally, I find that comment unfair. I have:
>        - Identified multiple show-stopper bugs in the routine
>        - Coded bug fixes
>        - Suggested and discussed extensions to make it usable on
>          multiple messages rather than running them together
>        - Coded the extension
> 
> I have shared my code the whole way through and have accepted input from
> you and Julian which ultimately made it better.

Agreed that you have done all of these.  But, after we have decided on 
a reasonable course of action (your proposal on 5 Jan, 23:50 UTC), 
things didn't go well.  You have posted the fixes here.  It wasn't a 
patch file.  So, I had to insert it manually into the code  base and 
test it.  When I tested it, it didn't work.  So I debugged it, found 
the fixes, committed them to the code base, and also posted a patch 
file for you (based on the code that you posted).  So, it is easy 
enough for you to test my version.  Did you test it?  Did you go back 
and retest your code posted here (on 6th Jan, 16:42 UTC)?

Your response was

    "Please try with the two patches I submitted. They really just
     correct probably typos. I don't think the changes you made are
     necessary. But please try my changes first..."

This is not helpful.  I have tried the "two patches you submitted", 
found problems, and sent you back the corrections.  Did you apply them 
and test them?  If you didn't do so, the further exchange didn't make 
any contribution.  In fact, it wasn't clear if we were even talking 
about the same things any more.

(My apologies for having misremembered that I committed the changes to 
launchpad.  I posted the patches here and was waiting to hear from 
you.  I have now uploaded my version to launchpad so that you can test 
it properly.)

I cannot be spending as much time on each and every issue.  If you 
want to make a contribution, please state your bug clearly and provide 
a *patchfile*, which I can apply and test, and after I accept it, we 
can move on.  I am also at liberty to revise your patches so as to 
maintain the consistency and coherence of the code base.  If you send 
me patches that fail my testing, that will make it harder for me to 
accept patches from you in future!  So, take the time to test them 
properly.  I am not going anywhere.

Cheers,
Uday
Technical Details
Message-ID: <hi73bg$1cs$1@north.jnrs.ja.net>
Path: arc1-ca.pugleaf.net!g2news1.google.com!news4.google.com!feeder.news-service.com!tudelft.nl!txtfeed1.tudelft.nl!dedekind.zen.co.uk!zen.net.uk!hamilton.zen.co.uk!feed4.jnfs.ja.net!feed2.jnfs.ja.net!jnfs.ja.net!times.reader.netnews.ja.net!not-for-mail
References: <m2d41p7sho.fsf@consult.pretender> <hht5o0$9pd$1@scotsman.ed.ac.uk> <m23a2l7myy.fsf@consult.pretender> <hhti24$9ua$1@scotsman.ed.ac.uk> <m2tyv15eaz.fsf@consult.pretender> <hhv7u9$bf4$1@scotsman.ed.ac.uk> <m2iqbg5zjg.fsf@consult.pretender> <hi0018$1gk$1@north.jnrs.ja.net> <m21vi45kxy.fsf@consult.pretender> <hi0e67$5b2$1@north.jnrs.ja.net> <m2tyv03yq9.fsf@consult.pretender> <m2pr5o3ycq.fsf@consult.pretender> <m2ljgc3xd3.fsf@consult.pretender> <hi2c72$l5i$1@north.jnrs.ja.net> <m2ocl7yvsr.fsf@consult.pretender> <hi4d25$81f$1@north.jnrs.ja.net> <m24omxzybc.fsf@consult.pretender> <hi54kr$euu$1@north.jnrs.ja.net> <m2r5q1ybz4.fsf@consult.pretender> <m2my0pybcx.fsf@consult.pretender> <hi5peg$kmp$1@north.jnrs.ja.net> <m2iqbdxbuj.fsf@consult.pretender>

Show Full Headers
Complete Article Headers
X-Google-Thread: a16957f451,747cd7667796b3e3
X-Google-Attributes: gida16957f451,public,usenet
X-Google-NewGroupId: yes
X-Google-Language: ENGLISH,ASCII-7-bit
Path: arc1-ca.pugleaf.net!g2news1.google.com!news4.google.com!feeder.news-service.com!tudelft.nl!txtfeed1.tudelft.nl!dedekind.zen.co.uk!zen.net.uk!hamilton.zen.co.uk!feed4.jnfs.ja.net!feed2.jnfs.ja.net!jnfs.ja.net!times.reader.netnews.ja.net!not-for-mail
From: Uday S Reddy <uDOTsDOTreddy@cs.bham.ac.uk>
Newsgroups: gnu.emacs.vm.bug
Subject: Re: Suggested fix to vm-pipe-message-part
Date: Fri, 08 Jan 2010 11:02:16 +0000
Organization: Janet Usenet Reading Service.
Lines: 56
Message-ID: <hi73bg$1cs$1@north.jnrs.ja.net>
References: <m2d41p7sho.fsf@consult.pretender> <hht5o0$9pd$1@scotsman.ed.ac.uk> <m23a2l7myy.fsf@consult.pretender> <hhti24$9ua$1@scotsman.ed.ac.uk> <m2tyv15eaz.fsf@consult.pretender> <hhv7u9$bf4$1@scotsman.ed.ac.uk> <m2iqbg5zjg.fsf@consult.pretender> <hi0018$1gk$1@north.jnrs.ja.net> <m21vi45kxy.fsf@consult.pretender> <hi0e67$5b2$1@north.jnrs.ja.net> <m2tyv03yq9.fsf@consult.pretender> <m2pr5o3ycq.fsf@consult.pretender> <m2ljgc3xd3.fsf@consult.pretender> <hi2c72$l5i$1@north.jnrs.ja.net> <m2ocl7yvsr.fsf@consult.pretender> <hi4d25$81f$1@north.jnrs.ja.net> <m24omxzybc.fsf@consult.pretender> <hi54kr$euu$1@north.jnrs.ja.net> <m2r5q1ybz4.fsf@consult.pretender> <m2my0pybcx.fsf@consult.pretender> <hi5peg$kmp$1@north.jnrs.ja.net> <m2iqbdxbuj.fsf@consult.pretender>
NNTP-Posting-Host: gromit.cs.bham.ac.uk
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-Trace: north.jnrs.ja.net 1262948528 1436 147.188.193.16 (8 Jan 2010 11:02:08 GMT)
X-Complaints-To: usenet@north.jnrs.ja.net
NNTP-Posting-Date: Fri, 8 Jan 2010 11:02:08 +0000 (UTC)
User-Agent: Thunderbird 2.0.0.23 (Windows/20090812)
In-Reply-To: <m2iqbdxbuj.fsf@consult.pretender>