Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Remove JSDOUBLE_COMPARE and content's equivalent DOUBLE_COMPARE

RESOLVED FIXED in mozilla12

Status

()

Core
JavaScript Engine
--
minor
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Waldo, Assigned: Waldo)

Tracking

unspecified
mozilla12
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 591373 [details] [diff] [review]
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.
Attachment #591373 - Flags: review?(dmandelin)

Comment 1

6 years ago
> 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... ;)
(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).
Attachment #591373 - Flags: review?(dmandelin) → review+
Oh, right, MSVC 2008 != 8.  Stupid non-version version numbers.

https://hg.mozilla.org/integration/mozilla-inbound/rev/c1321dc798b7
Target Milestone: --- → mozilla12

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/c1321dc798b7
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.