Closed Bug 6539 Opened 25 years ago Closed 25 years ago

[PP]nsFileSpec requires ResolveAlias() to be called on Mac

Categories

(Core :: XPCOM, defect, P3)

PowerPC
Mac System 8.0
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: waterson, Assigned: mcmullen)

Details

Using nsFileSpec on Mac requires use of ResolveAlias(). To write XP code, this requires, e.g., nsFileSpec spec(...); spec += "subdir"; #ifdef XP_MAC { PRBool wasAlias; spec.ResolveAlias(wasAlias); } #endif ...which is exactly the kind of code we're trying to avoid. IMHO :). Shouldn't these be resolved automatically? (Hey, soft links on Unix are...)
Status: NEW → ASSIGNED
Target Milestone: M7
Accepting for M7
Status: ASSIGNED → NEW
Summary: nsFileSpec requires ResolveAlias() to be called on Mac → [PP]nsFileSpec requires ResolveAlias() to be called on Mac
Assignee: warren → mcmullen
Why is this assigned to me?
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Because you own xpcom, and file i/o is a part of xpcom. Anyway, I fixed this today. Marking it fixed.
Did the ResolveAlias() method go away? Or is it still there, being called twice? In other words, will I need to go through and remove the #ifdef XP_MAC code at each call site that I alluded to above? thanks...
The "ResolveAlias()" method remains, and is called internally in a number of places now. After all outside calls have been removed, we might consider making this call protected, though. The only harm in calling it now is that it will be a NOOP that takes extra time that we can ill afford.
Bulk moving to XPCOM, in preparation for removal of XP File Handling component. (XPFH has received two bugs in ~5 months, and is no longer in active use.)
Component: XP File Handling → XPCOM
You need to log in before you can comment on or make changes to this bug.