cleanup xpcom/io for windows (reduce LoadLibrary/PR_LoadLibrary)

RESOLVED FIXED in mozilla12

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

Trunk
mozilla12
x86
Windows Vista
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

8 years ago
shell32.dll is already linked, so shell32.dll is loaded with xul.dll.  So we should use GetModuleHandleW instread.  This doesn't need to call FreeLibrary.

Also, nsLocalFileWin is for Windows only, it is no reason to use PR_LoadLibrary.
Assignee

Comment 1

8 years ago
Posted patch fixSplinter Review
Assignee

Updated

8 years ago
Attachment #587934 - Flags: review?(jmathies)

Updated

8 years ago
Attachment #587934 - Flags: review?(jmathies) → review+
https://hg.mozilla.org/mozilla-central/rev/3f378d354444
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.