From: Junio C Hamano <gitster@pobox.com>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 1/5] path-list: add functions to work with unsorted lists
Date: Thu, 28 Feb 2008 18:04:31 -0800 [thread overview]
Message-ID: <7vprugfrgw.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <alpine.LSU.1.00.0802290144410.22527@racer.site> (Johannes Schindelin's message of "Fri, 29 Feb 2008 01:44:56 +0000 (GMT)")
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> This is probably useful to a number of existing non-users of
> path_list.
I suspect most of the non-users will stay non-users, until the
thing is renamed to something more sensible.
The name gives a false impression that this is a special-purpose
data structure that is meant to handle pathnames very well
(e.g. canonicalizing "foo//../bar" to "bar" before doing
comparsion) but in reality it does not do anything like that,
and is a general purpose "table keyed with string" mechanism.
The wholesale renaming, if we were to do so, should happen when
things are quiet, and this is probably not a good time for it.
next prev parent reply other threads:[~2008-02-29 2:05 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-29 1:44 [PATCH 0/5] Builtin "git remote" Johannes Schindelin
2008-02-29 1:44 ` [PATCH 1/5] path-list: add functions to work with unsorted lists Johannes Schindelin
2008-02-29 2:04 ` Junio C Hamano [this message]
2008-02-29 2:15 ` Johannes Schindelin
2008-03-01 13:14 ` [PATCH 0/6] Builtin remote, v2 Johannes Schindelin
2008-03-01 13:15 ` [PATCH 1/6] Rename path_list to string_list Johannes Schindelin
2008-03-11 16:20 ` David Kågedal
2008-03-11 16:26 ` Johannes Schindelin
2008-03-11 16:44 ` David Kågedal
2008-03-01 13:15 ` [PATCH 2/6] string-list: add functions to work with unsorted lists Johannes Schindelin
2008-03-01 13:16 ` [PATCH 3/6] parseopt: add flag to stop on first non option Johannes Schindelin
2008-03-01 13:16 ` [PATCH 4/6] Test "git remote show" and "git remote prune" Johannes Schindelin
2008-03-01 13:16 ` [PATCH 5/6] Make git-remote a builtin Johannes Schindelin
2008-03-01 13:17 ` [PATCH 6/6] builtin-remote: prune remotes correctly that were added with --mirror Johannes Schindelin
2008-02-29 1:45 ` [PATCH 2/5] parseopt: add flag to stop on first non option Johannes Schindelin
2008-02-29 1:45 ` [PATCH 3/5] Test "git remote show" and "git remote prune" Johannes Schindelin
2008-02-29 1:45 ` [PATCH 4/5] Make git-remote a builtin Johannes Schindelin
2008-02-29 1:46 ` [PATCH 5/5] builtin-remote: prune remotes correctly that were added with --mirror Johannes Schindelin
2008-02-29 1:56 ` [PATCH 0/5] Builtin "git remote" Junio C Hamano
2008-02-29 2:13 ` Johannes Schindelin
2008-02-29 2:17 ` Daniel Barkalow
2008-02-29 11:21 ` Johannes Schindelin
2008-03-02 15:15 ` Johannes Schindelin
2008-03-02 15:40 ` Johannes Schindelin
2008-03-02 15:52 ` [PATCH 0/2] format-patch --cover-letter improvements Johannes Schindelin
2008-03-02 15:53 ` [PATCH 1/2] format-patch: use the diff options for the cover letter, too Johannes Schindelin
2008-03-02 15:53 ` [PATCH 2/2] format-patch: wrap cover-letter's shortlog sensibly Johannes Schindelin
2008-03-02 17:05 ` Junio C Hamano
2008-03-02 17:38 ` Johannes Schindelin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7vprugfrgw.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.