Article View: gnu.emacs.vm.bug
Article #297Re: Suggested fix to vm-pipe-message-part
From: Uday S Reddy
Date: Fri, 08 Jan 2010 11:02
Date: Fri, 08 Jan 2010 11:02
56 lines
4644 bytes
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-mailReferences:
<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>