Closed Bug 180316 Opened 22 years ago Closed 2 years ago

some files have too many #includes (not relevant to file)

Categories

(Core :: DOM: Editor, defect)

defect
Not set
normal

Tracking

()

RESOLVED INCOMPLETE
mozilla1.3beta

People

(Reporter: Brade, Unassigned)

Details

Attachments

(1 file, 2 obsolete files)

nsHTMLEditorStyle.cpp has lots of #includes of which many are not needed.  Other
files may have similar problems?
Attached patch remove some headers (obsolete) — Splinter Review
Attachment #107932 - Flags: superreview?(kin)
Attachment #107932 - Flags: review?(jfrancis)
Comment on attachment 107932 [details] [diff] [review]
remove some headers

sr=kin@netscape.com
Attachment #107932 - Flags: superreview?(kin) → superreview+
Attachment #107932 - Flags: review?(jfrancis) → review+
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.3beta
Attached patch include cleanup for composer/src (obsolete) — Splinter Review
Attachment #107932 - Attachment is obsolete: true
Attachment #127314 - Flags: superreview?(kin)
Attachment #127314 - Flags: review?(kaie)
Comment on attachment 127314 [details] [diff] [review]
include cleanup for composer/src

r=kaie
assuming you have tested it still compiles :-)
Attachment #127314 - Flags: review?(kaie) → review+
Comment on attachment 127314 [details] [diff] [review]
include cleanup for composer/src

sr=darin
Attachment #127314 - Flags: superreview?(kin) → superreview+
obsoleting patch that already landed
Attachment #127314 - Attachment is obsolete: true
Attachment #128006 - Flags: superreview?(tor)
Attachment #128006 - Flags: review?(kaie)
Comment on attachment 128006 [details] [diff] [review]
fix some includes in editor/idl

r=kaie
Attachment #128006 - Flags: review?(kaie) → review+
Attachment #128006 - Flags: superreview?(tor) → superreview+
QA Contact: sujay → editor

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: brade → nobody
Status: ASSIGNED → NEW

The bug report is based on too old revision. Please file a new bug if you find new including file issues in mozilla-central.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: