720941 - Remove JSDOUBLE_COMPARE and content's equivalent DOUBLE_COMPARE

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jeff Walden [:Waldo]]

Attachment: Patch

The implementation clearly indicates it was originally intended to work around some sort of Windows-only bug when it landed in 2004.  Whatever problem was being worked around also existed in 2006, when content/xslt/public/txDouble.h picked up a similar implementation. Six or eight years later, we don't even support anything before MSVC 2008 (obviously well after 2006), and tinderbox indicates it's no longer an issue (note that this is even the case with PGO builds, specifically enabled for the push).

https://tbpl.mozilla.org/?tree=Try&rev=f7cd9da8b89d

Let's get rid of these unnecessary, obfuscatory macros.

Comment 1

[Boris Zbarsky [:bzbarsky]]

> Six or eight years later, we don't even support anything before MSVC 2008

Tinderbox uses MSVC 2005.

But I'll believe we used something even older back in 2006... ;)

Comment 2

[David Mandelin [:dmandelin]]

(In reply to Jeff Walden (remove +bmo to email) from comment #0)
> Let's get rid of these unnecessary, obfuscatory macros.

Yes, let's. Especially if no rationale is given next to the macro definition: we either have to keep a weird thing forever that hardly any other software seems to need, or we can cut it out (and add it back in actually knowing why we need it on the off chance we actually do).

Comment 3

[Jeff Walden [:Waldo]]

Oh, right, MSVC 2008 != 8.  Stupid non-version version numbers.

https://hg.mozilla.org/integration/mozilla-inbound/rev/c1321dc798b7

Comment 4

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/c1321dc798b7