Closed Bug 1005321 Opened 12 years ago Closed 12 years ago

Convert the xpt code to C++

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla32

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(2 files)

Convert the xpt code to C++
15.38 KB, patch
bholley
: review+
Details | Diff | Splinter Review
Convert the xpt code to C++
17.52 KB, patch
bholley
: review+
Details | Diff | Splinter Review
No description provided.
Assignee: nobody → ehsan
Blocks: 994964
Flags: needinfo?(bobbyholley)
Flags: needinfo?(bobbyholley)
Attachment #8416720 - Flags: review?(bobbyholley)
Attachment #8416720 - Flags: review?(bobbyholley) → review+
Comment on attachment 8416750 [details] [diff] [review] Convert the xpt code to C++ Sorry, forgot the test files.
Attachment #8416750 - Flags: review?(bobbyholley)
Note to self: touch CLOBBER when landing this.
Comment on attachment 8416750 [details] [diff] [review] Convert the xpt code to C++ Review of attachment 8416750 [details] [diff] [review]: ----------------------------------------------------------------- I reviewed this on friday, but somehow didn't hit submit. :-(
Attachment #8416750 - Flags: review?(bobbyholley) → review+
Forgot to land the linker error fix that I had lying around for Windows: https://hg.mozilla.org/integration/mozilla-inbound/rev/038f364ab029
https://hg.mozilla.org/mozilla-central/rev/72fbb3c9a266 https://hg.mozilla.org/mozilla-central/rev/038f364ab029
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Depends on: 1006998
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: