Closed Bug 187130 Opened 22 years ago Closed 22 years ago

Remove #ifdef / #ifndef XP_OS2 in nsLocalFileOS2.cpp

Categories

(Core :: XPCOM, defect)

x86
OS/2
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Biesinger, Assigned: Biesinger)

Details

Attachments

(1 file)

nsLocalFileOS2.cpp is only built for OS/2 anyway, so the ifdefs seem unnecessary.

I'll attach a patch shortly.
Attached patch patchSplinter Review
While I were at it, I also removed an #if 0 block in nsLocalFileOS2.h, I hope
that's ok; and I also changed a NS_ENSURE_ARG to NS_ENSURE_ARG_POINTER because
it is a pointer.
Attachment #110337 - Flags: review?(mkaply)
Comment on attachment 110337 [details] [diff] [review]
patch

I built this and made sure it works.

Looks good.

Thanks for the help.
Attachment #110337 - Flags: review?(mkaply) → review+
Attachment #110337 - Flags: superreview?(bzbarsky)
Attachment #110337 - Flags: superreview?(bzbarsky) → superreview+
checked in
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: