Closed Bug 319402 Opened 20 years ago Closed 20 years ago

Simplify nsJSContext::CompileEventHandler call in nsEventListenerManager::AddScriptEventListener

Categories

(Core :: DOM: Events, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 obsolete file)

reading this function is hard enough, the extra confusion about unused and unnecessary params doesn't help my poor little mind.
Attached patch simplify function call (obsolete) — Splinter Review
Attachment #205206 - Flags: superreview?(jst)
Attachment #205206 - Flags: review?(jst)
Comment on attachment 205206 [details] [diff] [review] simplify function call r+sr=jst
Attachment #205206 - Flags: superreview?(jst)
Attachment #205206 - Flags: superreview+
Attachment #205206 - Flags: review?(jst)
Attachment #205206 - Flags: review+
Comment on attachment 205206 [details] [diff] [review] simplify function call mozilla/content/events/src/nsEventListenerManager.cpp 1.220
Attachment #205206 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Summary: Change nsJSContext::CompileEventHandler in nsEventListenerManager::AddScriptEventListener call to → Simplify nsJSContext::CompileEventHandler call in nsEventListenerManager::AddScriptEventListener
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: