Peter Maydell <pmaydell(_at_)chiark(_dot_)greenend(_dot_)org(_dot_)uk> wrote:
David Levine wrote:
I took a quick look at your dynamic allocation and it looks
fine to me
It hasn't got to me yet, but I had a look at it in the list archive, and
this line of the patch:
+ i +- namebufsiz;
looks very dubious to me :-)
Good catch. I haven't tested it yet, I need to make sure it works
properly when entering the realloc code.
It's also got at least one unrelated change in it:
+ if(uprf(cp, "re:") || uprf(cp, "fw:"))
Ah, part of my pending work on generalizing stripping of Re:, another
topic.
Other than that, it's not as invasive a patch as I'd feared it might be,
but I still vote to put it on ice until we have some decent tests.
Yes, testing good.
Can the OP send me one of the messages that had issues?
Thanks (谢谢).
Jeff
--
Jeffrey C. Honig <jch(_at_)honig(_dot_)net>
http://www.honig.net/jch
GnuPG ID:14E29E13 <http://www.honig.net/jch/key.shtml>
_______________________________________________
Nmh-workers mailing list
Nmh-workers(_at_)nongnu(_dot_)org
http://lists.nongnu.org/mailman/listinfo/nmh-workers