1. 20 Jun, 2021 1 commit
  2. 19 Jun, 2021 1 commit
    • Kevin J. McCarthy's avatar
      Fix menu.c sigwinch handling order. · f5ccd687
      Kevin J. McCarthy authored
      This fixes problems with a sigwinch that occurs during an external
      process, such as during a query menu lookup, or after exiting the
      background-edit landing page.  Without this change, the user would see
      a misdrawn screen and would need to press a key first.
      
      Change it so that the SigWinch flag is handled before reading input,
      so that the page is automatically redrawn.  This matches the behavior
      in the index, pager, and background-edit process list menu.
      f5ccd687
  3. 18 Jun, 2021 1 commit
    • Kevin J. McCarthy's avatar
      Reset SIGPIPE signal handler in child process before exec(). · 80ed1d85
      Kevin J. McCarthy authored
      Ignored signals, such as SIGPIPE in Mutt, are inherited by child
      processes.  This can cause incorrect behavior in filter-like programs
      that expect SIGPIPE to be the default value of "Term".
      
      A couple other places in Mutt already reset SIGTERM, SIGTSTP, and
      SIGCONT, with a comment mentioning it really wasn't necessary.  Since
      I don't much like messing with signals, I'll leave those resets in.
      
      Extract those plus SIGPIPE into a new function,
      mutt_reset_child_signals() and add that before each exec call.
      
      Thanks to Vincent Lefèvre for reporting the issue and for the initial
      patch, which this commit is based upon.
      80ed1d85
  4. 17 Jun, 2021 2 commits
    • Rene Kita's avatar
      Add $pager_skip_quoted_context option. · 9155c657
      Rene Kita authored
      Using <skip-quoted> the first unquoted line becomes the new top line
      displayed in the pager. This leaves the user with no context to know
      what the answer refers to.
      
      Add an option to keep some lines of context when using <skip-quoted>.
      
      With $pager_skip_quoted_context set to 5, <skip-quoted> will show at
      most 5 lines of the previous quote. If the previous quote is shorter
      than 5 lines the whole quote will be displayed.
      
      This option defaults to 0 to remain backwards compatible.
      9155c657
    • Vincent Lefevre's avatar
      Updated French translation. · 4595cdb6
      Vincent Lefevre authored
      4595cdb6
  5. 13 Jun, 2021 4 commits
    • Kevin J. McCarthy's avatar
      Suggested translation changes. · 86700b01
      Kevin J. McCarthy authored
      These were forwarded to mutt-po by Helge Kreutzmann, from his
      translation team.
      
      I've added comments to a couple places where the meaning was unclear.
      86700b01
    • Kevin J. McCarthy's avatar
      Merge branch 'stable' · 94d0285c
      Kevin J. McCarthy authored
      94d0285c
    • Kevin J. McCarthy's avatar
      Fix getkeybyaddr() debug output. · e1dfc2c4
      Kevin J. McCarthy authored
      Add a NONULL wrapper and a newline at the end.
      e1dfc2c4
    • Kevin J. McCarthy's avatar
      Directly add full mailbox to GPG search hints. · 3cf7a149
      Kevin J. McCarthy authored
      The code previously called (crypt/pgp)_add_string_to_hints(), which
      performs a strtok() using " ,.:\"()<>\n" and discards parts of size 3
      or less.
      
      This makes sense for personal fields, and random "stuff" entered in
      the getkeybystr() prompt.  But it doesn't make a lot of sense for the
      mailbox.
      
      There's no advantage I can see in chopping off and discarding the
      domain name for getting more relevant results.
      
      Ordinarily, I wouldn't bother fixing this, but in ticket 354 someone
      is experimenting with alternative backends, and they would like the
      full email address listed in the hints.
      3cf7a149
  6. 12 Jun, 2021 3 commits
  7. 07 Jun, 2021 2 commits
  8. 05 Jun, 2021 2 commits
  9. 03 Jun, 2021 1 commit
  10. 31 May, 2021 1 commit
  11. 30 May, 2021 1 commit
  12. 29 May, 2021 3 commits
  13. 28 May, 2021 5 commits
    • Flammie Pirinen's avatar
      Update Finnish translation · 3d24855d
      Flammie Pirinen authored
      3d24855d
    • Eike Rathke's avatar
      Do not set From reverse name on postponed or to be resend messages · 36fb911f
      Eike Rathke authored
      Messages to be resend or resumed from postponed already had
      reverse names and alternates evaluated the first time they entered
      the compose menu and send-hook were already executed then. The
      user may also had edited the From to any other content before
      postponing or sending the message. Upon resuming, do not overwrite
      with a reverse_name lookup and do not remove the real name if
      reverse_realname=no is set.
      
      This restores the old behaviour and fixes the regression
      introduced with
      
          commit 961cf4be
          CommitDate: Mon Feb 1 14:00:56 2021 -0800
      
              Apply $reverse_name when tag-reply/forwarding too.
      36fb911f
    • Emir Sarı's avatar
      Update Turkish translations · ddd292b8
      Emir Sarı authored
      ddd292b8
    • Greg Kroah-Hartman's avatar
      fix gcc-11 build warnings for H_TO_INTL() · 88845a41
      Greg Kroah-Hartman authored
      The macro H_TO_INTL() in mutt_idna.c, causes a lot of build warnings
      about the indentation of the if () line to try to warn the developer
      that it looks like this might be a bug:
      
      mutt_idna.c: In function ‘mutt_env_to_intl’:
      mutt_idna.c:327:5: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
        327 |     if (tag) *tag = #a; e = 1; err = NULL;              \
            |     ^~
      mutt_idna.c:333:3: note: in expansion of macro ‘H_TO_INTL’
        333 |   H_TO_INTL(return_path);
            |   ^~~~~~~~~
      mutt_idna.c:327:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
        327 |     if (tag) *tag = #a; e = 1; err = NULL;              \
            |                         ^
      mutt_idna.c:333:3: note: in expansion of macro ‘H_TO_INTL’
        333 |   H_TO_INTL(return_path);
            |   ^~~~~~~~~
      
      This goes on for many lines.
      
      Fix this up by properly indenting the lines so that gcc "knows" we mean
      well here.
      88845a41
    • Maxim Tarasov's avatar
      Trim the ToC for the option list in the manual · 76276257
      Maxim Tarasov authored
      Currently, every variable is shown in the table of contents of the
      reference manual. Variable list is over 400 items long. This makes the
      ToC difficult to work with.
      
      Removing is done by adding an overwriting template for the parent of
      variables section (that is sect1 tag with id="variables"). In the
      DocBook source matching template calls another template named subtoc
      with two parameters: context (self element) and nodes (child
      sections). Normally a list of descendant nodes is passed here, but we
      don't want that and instead call subtoc with no parameters.
      
      This approach results in semantically correct code being generated in
      all modes.
      
      Note, that the ID (variables) will have to be kept in sync with XML
      generating code, but that is already hardcoded elsewhere.
      76276257
  14. 26 May, 2021 2 commits
  15. 23 May, 2021 2 commits
  16. 15 May, 2021 1 commit
    • Gregory Anders's avatar
      Add local_date_header option · 4c786d87
      Gregory Anders authored
      Add an option to convert the date used in the Date header into the local
      (sender's) timezone. This is the current behavior and the option
      defaults to true. Unsetting this option causes the date in the Date
      header to be formatted using the GMT timezone.
      
      This option is useful for privacy-sensitive users who may not wish to
      divulge their sending timezone.
      4c786d87
  17. 04 May, 2021 4 commits
  18. 03 May, 2021 1 commit
    • Kevin J. McCarthy's avatar
      Fix seqset iterator when it ends in a comma. · 7c4779ac
      Kevin J. McCarthy authored
      If the seqset ended with a comma, the substr_end marker would be just
      before the trailing nul.  In the next call, the loop to skip the
      marker would iterate right past the end of string too.
      
      The fix is simple: place the substr_end marker and skip past it
      immediately.
      7c4779ac
  19. 01 May, 2021 3 commits
    • Kevin J. McCarthy's avatar
      b24a1a25
    • Kevin J. McCarthy's avatar
      Add sanitize call in print mailcap function. · a10d0bb5
      Kevin J. McCarthy authored
      The mutt_rfc1524_expand_filename() function calls mutt_adv_mktemp(),
      which also calls the sanitizer.
      
      However, if the recv-mode suggested filename has a slash, it's better
      to sanitize before calling expand_filename() too, to preserve the
      entire filename (albeit with the slashes sanitized out).
      
      For example (ignoring name templates) if the file were called
      "either/or.pdf", pre-sanitizing would generate "either_or.pdf", while
      not doing so would generate "or.pdf".
      a10d0bb5
    • Kevin J. McCarthy's avatar
      Add allow_8bit flag to sanitize_filename functions. · 98f0ca6b
      Kevin J. McCarthy authored
      This will allow "view" mailcap invocations and autoview invocations to
      display something reasonable for international file names.  Also add
      the flag to mutt_adv_mktemp(), which the mailcap viewer calls.
      
      Convert the 'slash parameter' to an ALLOW_SLASH flag.  This makes the
      flags consistent (both "allow" something not permitted by default),
      and invocations more readable.
      98f0ca6b