Closed Bug 1114052 Opened 6 years ago Closed 6 years ago

Don't include Windows headers inside a namespace.

Categories

(Toolkit :: Downloads API, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: jacek, Assigned: jacek)

Details

Attachments

(1 file)

Attached patch patch.diffSplinter Review
The comment about mozIStorageError seems outdated. It builds fine then included on top of the file:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4f98e127a706

Those headers are included in namespace now. This causes problems with mingw builds.
Attachment #8539708 - Flags: review?(paolo.mozmail)
Attachment #8539708 - Flags: review?(paolo.mozmail) → review+
https://hg.mozilla.org/mozilla-central/rev/ec8e54c0352e
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.