Closed Bug 17781 Opened 25 years ago Closed 22 years ago

Clean up:remove xp_mcom complications on mac(in gif.cpp and xpcompat.h)

Categories

(Core :: Graphics: ImageLib, defect, P3)

PowerPC
Mac System 8.5
defect

Tracking

()

VERIFIED FIXED
Future

People

(Reporter: pnunn, Assigned: scc)

References

Details

NET_BACat in now in gif.cpp, the only place it is used. It is renamed IL_BACat. **Change BlockAllocCat to BlkAllocCat to prevent collision with old code. Trace why xp_mcom.h needed in src/xpcompat.h. Why is xp_mcom.h needed in dllcompat.h for all decoders.
Status: NEW → ASSIGNED
Target Milestone: M12
Target Milestone: M12 → M13
*** Bug 3791 has been marked as a duplicate of this bug. ***
*** Bug 8520 has been marked as a duplicate of this bug. ***
gif part done. Still working on the xpcompat.h part. -pn
Target Milestone: M13 → M14
I can push the xpcompat.cpp changes to m14. -p
pushing to m16. This is cleanup, not a bug or additional feature. not urgent. -pn
Target Milestone: M14 → M16
Moving to m17
Target Milestone: M16 → M17
Scott, As per discussion of this bug with dp, Pam and myself, dp suggested that you could help out Pam on the mac, to remove xp_mcom complications on mac(in gif.cpp and xpcompat.h). Hence reassigning to you ar per dp. Thanks, Neeti
Assignee: pnunn → scc
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Target Milestone: M17 → Future
mass re-assigning to my new bugzilla account
Assignee: scc → scc
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
QA Contact: elig → tpreston
Depends on: 70938
Changing platform
Hardware: PC → Macintosh
URL: n/a
http://lxr.mozilla.org/seamonkey/find?string=xpcompat.h shows that no file named xpcompat.h exists [anymore] chb@frodo:/part2/mozilla/modules/libimg/gifcom$ grep -r xp_ * chb@frodo:/part2/mozilla/modules/libimg/gifcom$ chb@frodo:/part2/mozilla/modules/libpr0n/decoders/gif$ grep -r xp_ * chb@frodo:/part2/mozilla/modules/libpr0n/decoders/gif$ looks like xp_mcom isn't used in the gif decoder. can this bug be closed now?
closing as per comment above
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
verified per comment 10
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.