Re: [PATCH] MSVC: port pthread code to native Windows threads

From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: "Andrzej K. Haczewski" <ahaczewski@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] MSVC: port pthread code to native Windows threads
Date: Wed, 4 Nov 2009 15:04:09 +0100 (CET)	[thread overview]
Message-ID: <alpine.DEB.1.00.0911041427140.2788@felix-maschine> (raw)
In-Reply-To: <1257331059-26344-1-git-send-email-ahaczewski@gmail.com>

Hi,

On Wed, 4 Nov 2009, Andrzej K. Haczewski wrote:

> diff --git a/builtin-pack-objects.c b/builtin-pack-objects.c
> index 02f9246..c96d293 100644
> --- a/builtin-pack-objects.c
> +++ b/builtin-pack-objects.c
> @@ -2327,6 +2327,18 @@ int cmd_pack_objects(int argc, const char **argv, const char *prefix)
>  #ifdef THREADED_DELTA_SEARCH
>  	if (!delta_search_threads)	/* --threads=0 means autodetect */
>  		delta_search_threads = online_cpus();
> +
> +#ifdef _WIN32

This flies in the face of our endeavors to enhance readability by reducing 
the number of #ifdef's, and at least guarding the #ifdef'ed parts behind 
meaningful names rather than platform specifiers.

See for example THREADED_DELTA_SEARCH: it does not read "HAS_PTHREADS" or 
some such.

Ciao,
Dscho