Re: [PATCH 2/3] rev-list: Introduce --no-output to avoid /dev/null redirects
- From
Shawn O. Pearce <spearce@spearce.org>
- Date
- Nov 9, 2007, 09:12 UTC
- Message-ID
- <20071109091219.GQ14735@spearce.org>
- In-Reply-To
- <7vsl3fvlrb.fsf@gitster.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> wrote:
Show 13 quoted lines
> Alex Riesen <raa.lkml@gmail.com> writes: > > Junio C Hamano, Fri, Nov 09, 2007 08:32:01 +0100: > >> > >> The function names noshow_xxx() looked a bit funny, but I do not > >> offhand have better alternatives to offer. > > > > "hide", "skip", "ignore"? > > But look at what the functions do. The original show_xxx() was > to print and then process. Shawn splitted them into show_xxx() > and noshow_xxx(), leaft the printing part in the former, made > the former call the latter at the end, and moved the processing > to the latter. So it is not any of the three words.
In my latest patch series I've gone with "finish_xxx()" as it is finishing our usage of that object. I'll be resubmitting a new series soon.
-- Shawn.