#3837: [PATCH] Add width expando for external filters
--------------------------+----------------------
  Reporter:  flatcap      |      Owner:  mutt-dev
      Type:  enhancement  |     Status:  new
  Priority:  minor        |  Milestone:
 Component:  display      |    Version:  1.6.0
Resolution:               |   Keywords:  patch
--------------------------+----------------------

Comment (by kevin8t8):

 Thanks for the patch.  I have to think about this.  Mailcap is a
 standardized file (RFC1524) that isn't exclusive to mutt.

 While the patch is easy enough to understand, I don't know that we want to
 encourage non-standard modification of the mailcap file.

 Some nits for the patch itself:
 - The second param for snprintf should probably be (sizeof (buf) - y), not
 6.
 - The return value for snprintf needs to be checked carefully.  (It may
 return a value larger than the size passed in, to indicate truncation)
 - The comment for the non-standard param doesn't belong in the
 "Unsupported rfc1524 parameters" list
 - We would probably want to use MuttIndexWindow->cols instead of COLS.

--
Ticket URL: <https://dev.mutt.org/trac/ticket/3837#comment:1>
Mutt <http://www.mutt.org/>
The Mutt mail user agent

Reply via email to