Closed Bug 707498 Opened 8 years ago Closed 8 years ago

Use a template to get the array length for nsGenericElement::FindAttributeDependence

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

Details

Attachments

(1 file)

Comment on attachment 578896 [details] [diff] [review]
Patch v1

Review of attachment 578896 [details] [diff] [review]:
-----------------------------------------------------------------

Looks nice.  Is this every caller of FindAttributeDependence?  If it is, please mark the original FindAttributeDependence function private within nsGenericElement.  If it isn't, we should consider whether the other callers could be changed somehow.  I'll ping you on IRC tomorrow and we can figure out something here, if something need be figured out.
Attachment #578896 - Flags: review?(jwalden+bmo) → review+
https://hg.mozilla.org/mozilla-central/rev/c74e2c35fd96
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.