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

Re: [PATCH next 2/2] Remove python 2.5'isms

From
Sverre Rabbelier <srabbelier@gmail.com>
Date
Jun 10, 2010, 07:35 UTC
Message-ID
<AANLkTimLiLfGAMLp0n86eCIQFViu4veu0aTL6yuA_8f1@mail.gmail.com>
In-Reply-To
<4C1094BF.5070907@drmicha.warpmail.net>
Heya,

On Thu, Jun 10, 2010 at 09:31, Michael J Gruber <git@drmicha.warpmail.net> wrote:

> Note that due to the use of an iterator (which P2.4 has) rather than a
> list or set, the expression is just as efficient, i.e. calls
> startswith() only as often as necessary.

I like that version a lot better, thanks. And I agree, the check_call replacement code should just be put into it's own function in a similar way, and we should use subprocess if it is available.

-- 
Cheers,

Sverre Rabbelier
Previous: Michael J Gruber
Message 12 of 12 in “Makefile: add PYTHON_PATH to GIT-BUILD-OPTIONS”
  1. 1/2 Makefile: add PYTHON_PATH to GIT-BUILD-OPTIONSBrandon Casey, Jun 9, 2010
  2. 2/2 Remove python 2.5'ismsBrandon Casey, Jun 9, 2010
  3. Sverre RabbelierJun 9, 2010
  4. Johan HerlandJun 9, 2010
  5. Sverre RabbelierJun 9, 2010
  6. 2/2 Remove python 2.5'ismsBrandon Casey, Jun 10, 2010
  7. Sverre RabbelierJun 10, 2010
  8. Michael J GruberJun 10, 2010
  9. David RiptonJun 10, 2010
  10. Michael J GruberJun 11, 2010
  11. Michael J GruberJun 10, 2010
  12. Sverre RabbelierJun 10, 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.