From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=45690 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1PZB9O-0008NF-Ta
	for qemu-devel@nongnu.org; Sat, 01 Jan 2011 18:46:19 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1PZB9N-0004Wv-PS
	for qemu-devel@nongnu.org; Sat, 01 Jan 2011 18:46:18 -0500
Received: from mail-vw0-f45.google.com ([209.85.212.45]:63462)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1PZB9N-0004Wr-Mz
	for qemu-devel@nongnu.org; Sat, 01 Jan 2011 18:46:17 -0500
Received: by vws12 with SMTP id 12so5048367vws.4
	for <qemu-devel@nongnu.org>; Sat, 01 Jan 2011 15:46:16 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <1292601366-990-1-git-send-email-peter.maydell@linaro.org>
References: <1292601366-990-1-git-send-email-peter.maydell@linaro.org>
Date: Sat, 1 Jan 2011 23:46:16 +0000
Message-ID: <AANLkTi=xQtNywQftT1kDGZROBppQ+P67yKrS5a1ZFkgi@mail.gmail.com>
Subject: Re: [Qemu-devel] [PATCH V2] softfloat: Rename float*_is_nan()
	functions to float*_is_quiet_nan()
From: Peter Maydell <peter.maydell@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: qemu-devel@nongnu.org
Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Aurelien Jarno <aurelien@aurel32.net>, Nathan Froyd <froydnj@codesourcery.com>

On 17 December 2010 15:56, Peter Maydell <peter.maydell@linaro.org> wrote:
> The softfloat functions float*_is_nan() were badly misnamed,
> because they return true only for quiet NaNs, not for all NaNs.
> Rename them to float*_is_quiet_nan() to more accurately reflect
> what they do.
>
> This change was produced by:
> =C2=A0perl -p -i -e 's/_is_nan/_is_quiet_nan/g' $(git grep -l is_nan)
> (with the results manually checked.)
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> Reviewed-by: Nathan Froyd <froydnj@codesourcery.com>
> Acked-by: Edgar E. Iglesias <edgar.iglesias@gmail.com>

Ping? This patch got no further comments (although it provoked
a long thread on a completely unrelated topic), it's been
reviewed and it still applies to master currently. I think it's
an uncontroversially good idea (~80% of current uses of the
functions are actually buggy because the people who wrote
them were misled by the function name!).

Can it be applied please? (cc'd Aurelien since you seem to be
committing various missed patches at the moment :-))

thanks
-- PMM