Macro for FIND_MAP_AND_CALL_METHOD_ON_IT

NEW
Unassigned

Status

()

Core
Layout: Tables
11 years ago
11 years ago

People

(Reporter: Bernd, Unassigned)

Tracking

(Depends on: 1 bug)

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

11 years ago
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
You need to log in before you can comment on or make changes to this bug.