Closed
Bug 920061
Opened 12 years ago
Closed 12 years ago
Use explicit template instantiations instead of a hack
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: sunfish, Assigned: sunfish)
Details
Attachments
(1 file)
|
4.73 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
I just noticed the hack from bug 915998 and it turns out that there's a standard C++ way to do this. The attached patch implements it.
Attachment #809209 -
Flags: review?(n.nethercote)
Comment 1•12 years ago
|
||
I swear I tried that, or something like it, at one point.
r=me if try server says it's ok. Thanks!
| Assignee | ||
Comment 2•12 years ago
|
||
Comment 3•12 years ago
|
||
Comment on attachment 809209 [details] [diff] [review]
explicit-instantiate.patch
Review of attachment 809209 [details] [diff] [review]:
-----------------------------------------------------------------
Well, it didn't get backed out, so I'm assuming that this technique works fine. r=me
Attachment #809209 -
Flags: review?(n.nethercote) → review+
| Assignee | ||
Comment 4•12 years ago
|
||
Oops, I interpreted your earlier r=me as a review approval, conditional on a try run. I had collected what I judged to be sufficient try run data, so I checked it in. Thanks for your actual review approval :-}.
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Comment 6•12 years ago
|
||
> Oops, I interpreted your earlier r=me as a review approval, conditional on a
> try run.
That's what I meant... it appears I just forgot to change the checkbox to a '+' :) Thanks for intervening, Terrence.
Comment 7•12 years ago
|
||
(In reply to Dan Gohman [:sunfish] from comment #4)
> Oops, I interpreted your earlier r=me as a review approval, conditional on a
> try run. I had collected what I judged to be sufficient try run data, so I
> checked it in.
Usually we'd dump the try-run URL in here, so it's obvious that's happened. :-) Even if it is a partial run -- I assume people will see that "enough" confirmation was gotten in that case.
You need to log in
before you can comment on or make changes to this bug.
Description
•