Closed Bug 491591 Opened 16 years ago Closed 13 years ago

Move ns{Int,}{Point,Size,Margin,Rect} into namespace mozilla

Categories

(Core :: Graphics, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: zwol, Unassigned)

References

Details

The classes listed in the summary should move into namespace mozilla and lose the ns-prefix. Post bug 470506, their base classes are already in namespace mozilla. This change is straightforward but requires search-and-replace fixups through a lot of code (will probably be done with pork). We might want to rename the classes at the same time - thinking about this pattern: nsIntPoint -> mozilla::PxPoint nsPoint -> mozilla::AuPoint The prefixes encode the units of the coordinates stored in each type: 'Px' is for CSS pixels, 'Au' is for app units (a bit obscure, alas). Am open to better ideas.
No longer depends on: 470506
Au is probably bad since it is an actual unit
It is? The only thing it means to me is the chemical symbol for gold, but that's clearly irrelevant in this context...
1au = 1.49598e14 millimeters
i've been handed a clue on irc: astronomical units, of course. suggestions for better name very welcome.
what's wrong with "AppPoint" (AppRect/AppSize)? I don't really like PxPoint either, since fractional pixels are just fine.. I'd keep that as IntPoint, since the type is the explicit piece there that's important.
Depends on: 470506
I no longer work for Mozilla. Deassigning myself from bugs I don't intend to work on as a volunteer.
Assignee: zackw → nobody
Status: ASSIGNED → NEW
We now have mozilla::gfx::{Int,}{Point,Size,Margin,Rect}; I don't know that there's a lot of benefit to changing legacy nsRect et al.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.