git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 00/16] git-send-email cleanups

From
Brian Gernhardt <brian@gernhardtsoftware.com>
Date
Sep 30, 2010, 14:30 UTC
Message-ID
<18E0A903-D625-4C7A-A575-AC5C5EF448C9@gernhardtsoftware.com>
In-Reply-To
<1285854189-10240-1-git-send-email-avarab@gmail.com>
On Sep 30, 2010, at 9:42 AM, Ævar Arnfjörð Bjarmason wrote:
>  send-email: get_patch_subject doesn't need a prototype
>  send-email: file_declares_8bit_cte doesn't need a prototype
>  send-email: unique_email_list doesn't need a prototype
>  send-email: cleanup_compose_files doesn't need a prototype
None of these subroutines strictly need the prototype, but it does allow Perl to warn us if we send incorrect arguments.  Why remove them?  Are they causing problems somewhere?
~~ Brian
Previous: Ævar Arnfjörð BjarmasonNext: Jeff King
Message 24 of 29 in “git-send-email cleanups”
  1. 00/16 git-send-email cleanupsÆvar Arnfjörð Bjarmason, Sep 30, 2010
  2. 01/16 send-email: use lexical filehandle for opendirÆvar Arnfjörð Bjarmason, Sep 30, 2010
  3. 02/16 send-email: use lexical filehandles for $composeÆvar Arnfjörð Bjarmason, Sep 30, 2010
  4. 03/16 send-email: use lexical filehandles during sendingÆvar Arnfjörð Bjarmason, Sep 30, 2010
  5. 04/16 send-email: get_patch_subject doesn't need a prototypeÆvar Arnfjörð Bjarmason, Sep 30, 2010
  6. 05/16 send-email: file_declares_8bit_cte doesn't need a prototypeÆvar Arnfjörð Bjarmason, Sep 30, 2010
  7. 06/16 send-email: unique_email_list doesn't need a prototypeÆvar Arnfjörð Bjarmason, Sep 30, 2010
  8. 07/16 send-email: cleanup_compose_files doesn't need a prototypeÆvar Arnfjörð Bjarmason, Sep 30, 2010
  9. 08/16 send-email: use \E***\Q instead of \*\*\*Ævar Arnfjörð Bjarmason, Sep 30, 2010
  10. 09/16 send-email: sanitize_address use $foo, not "$foo"Ævar Arnfjörð Bjarmason, Sep 30, 2010
  11. 10/16 send-email: sanitize_address use qq["foo"], not "\"foo\""Ævar Arnfjörð Bjarmason, Sep 30, 2010
  12. 11/16 send-email: use (?:) instead of () if no match variables are neededÆvar Arnfjörð Bjarmason, Sep 30, 2010
  13. 12/16 send-email: is_rfc2047_quoted use qr// regexesÆvar Arnfjörð Bjarmason, Sep 30, 2010
  14. 13/16 send-email: extract_valid_address use qr// regexes and /oÆvar Arnfjörð Bjarmason, Sep 30, 2010
  15. Jeff KingSep 30, 2010
  16. Ævar Arnfjörð BjarmasonSep 30, 2010
  17. Jeff KingOct 1, 2010
  18. Junio C HamanoSep 30, 2010
  19. Ævar Arnfjörð BjarmasonSep 30, 2010
  20. 13/16 send-email: extract_valid_address use qr// regexesÆvar Arnfjörð Bjarmason, Sep 30, 2010
  21. 14/16 send-email: send_message die on $!, not $?Ævar Arnfjörð Bjarmason, Sep 30, 2010
  22. 15/16 send-email: make_message_id use "require" instead of "use"Ævar Arnfjörð Bjarmason, Sep 30, 2010
  23. 16/16 send-email: use Perl idioms in while loopÆvar Arnfjörð Bjarmason, Sep 30, 2010
  24. Brian GernhardtSep 30, 2010
  25. Jeff KingSep 30, 2010
  26. Brian GernhardtSep 30, 2010
  27. Ævar Arnfjörð BjarmasonSep 30, 2010
  28. Jay SoffianSep 30, 2010
  29. Jeff KingSep 30, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.