Re: [Nmh-workers] Re: [Nmh-commits] nmh ChangeLog uip/mhparse.c test/tests/mhshow/t...

[Eric Gillespie]

[Note: I don't see a message on nmh-commits for the de-dup change
 I discuss here; you may have to update your working copy to see
 what I'm talking about.]

[Note: the "PatchSet"s I refer to are from

  CVSROOT=:pserver:address@hidden:/sources/nmh TZ=UTC cvsps -u -A nmh

 It's the only thing we have to refer to a change across more than
 one file.]

Eric Gillespie <address@hidden> writes:

> I have two more Postel's Law fixes in my tree.  The next fix to

Done.

> pull from my tree is the unification of mhparse.c and
> mhbuildsbr.c.  I'd forgotten all about that nastiness...

Aside from those listed in the ChangeLog, I noted the following
divergences between mhparse.c and mhbuildsbr.c's copy:

- An rfc934sw check in mhbuildsbr.c's copy of InitMultiPart that
  is harmless to leave out; the remaining checks in mhbuildsbr.c
  are sufficient.

- get_content in mhparse.c first parses all the headers into a
  linked list and then loops over that looking for interesting
  headers (though it left out Content-Disposition, added in
  PatchSet 549, when support to create those was added to
  mhbuild).  That's a better approach, as it's not repeating the
  header parsing logic over and over again, as attested by
  diffstat: 187 insertions(+), 224 deletions(-)

---------------------
PatchSet 138 
Date: 2000/03/13 18:29:26
Author: danh
Branch: HEAD
Tag: (none) 
Log:
Applied Sullivan N. Beck <address@hidden>'s mhshow-suffix patch:

    With the patch below, you can add lines like:

        mhshow-suffix-application/msword: .doc
        mhshow-suffix-application/PostScript: .ps

    to the mhn.defaults file to append the given suffix to a scratch file.  This
    allows applications which require a certain suffix to run properly.

Removed -force_html from lynx entry in mhn.defaults.sh (I believe older versions
of lynx lack that option) and added "mhshow-suffix-text/html: .html".

Members: 
        ChangeLog:1.47->1.48 
        etc/mhn.defaults.sh:1.4->1.5 
        man/mhn.man:1.1->1.2 
        man/mhshow.man:1.1->1.2 
        uip/mhparse.c:1.4->1.5 

---------------------
PatchSet 439 
Date: 2003/09/21 08:43:00
Author: gbburkhardt
Branch: HEAD
Tag: (none) 
Log:
Apply bug fix reported to Debian (#136976) by Peter Maydell; allows
messages with "Content-Transfer-Encoding: binary" to be
displayed, per RFC 2045 section 6.2 paragraph 4

Members: 
        uip/mhparse.c:1.9->1.10 

---------------------
PatchSet 497 
Date: 2005/11/05 16:32:20
Author: pm215
Branch: HEAD
Tag: (none) 
Log:
Don't fclose() the file twice in error-exit paths from get_content()
(was causing crashes on multipart mails with invalid Content-Type headers)

Members: 
        ChangeLog:1.190->1.191 
        uip/mhparse.c:1.11->1.12 

---------------------
PatchSet 549 
Date: 2006/02/18 08:26:37
Author: levine
Branch: HEAD
Tag: (none) 
Log:
Added support for optional Content_Disposition header in mhbuild directive.s

Members: 
        ChangeLog:1.233->1.234 
        docs/TODO:1.3->1.4 
        h/mhparse.h:1.1->1.2 
        h/mime.h:1.1->1.2 
        man/mhbuild.man:1.7->1.8 
        uip/mhbuildsbr.c:1.11->1.12 
        uip/mhfree.c:1.4->1.5 

-- 
Eric Gillespie <*> address@hidden