Open Bug 355940 Opened 18 years ago Updated 2 years ago

Macro for FIND_MAP_AND_CALL_METHOD_ON_IT

Categories

(Core :: Layout: Tables, defect)

Product:
Core
Component:
Layout: Tables
Platform:
x86
Windows XP
Type:
defect

Tracking

()

People

(Reporter: bernd_mozilla, Unassigned)

References

(Depends on 1 open bug)

Details

followup to bug 351942
>Index: nsCellMap.cpp

> {
>   PRInt32 rowIndex = aRowIndex;
>   nsCellMap* map = mFirstMap;
>   while (map) {
>     if (map->GetRowCount() > rowIndex) {
...
>     }
>     rowIndex -= map->GetRowCount();
>     map = map->GetNextSibling();
>   }

This code pattern is repeated a _lot_.  It's worth filing a followup bug to
make it into a macro.  Perhaps something like FIND_MAP_AND_CALL_METHOD_ON_IT or
something....
*** Bug 357110 has been marked as a duplicate of this bug. ***
I just realized that the big patch in bug 356335 effectively implements this... So I guess I'll do this in bug 357729 (where most of the big patch will go).
Depends on: 357729
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.