Closed Bug 224623 Opened 18 years ago Closed 11 years ago

Remove obsolete files from extensions/spellcheck

Categories

(Core :: Spelling checker, defect)

defect
Not set
minor

Tracking

()

RESOLVED DUPLICATE of bug 339106

People

(Reporter: mvl, Unassigned)

Details

Attachments

(1 file)

There are a few unused files in extensions/spellcheck/src:

nsSpellCheckController.cpp
nsSpellCheckController.h
nsSpellCheckModule.cpp
nsSpellCheckModule.h
nsSpellCheckUtils.cpp
nsSpellCheckUtils.h

They should be removed, to reduce confusion. They became unused when the
spellchecker went in to the default builds.
Are we sure that rods will never finish this work?
if he ever wants to, he need but look in bonsai :)
Component: Editor: Core → Spelling checker
QA Contact: bugzilla → core.spelling-checker
Attachment #134924 - Flags: superreview?(mscott)
Attachment #134924 - Flags: review?(timeless)
Comment on attachment 134924 [details] [diff] [review]
patch to remove the  files

what about:

http://lxr.mozilla.org/mozilla/source/extensions/spellcheck/src/nsSpellCheckMod
ule.cpp#48

which references spellcheckccontroller?
The patch removes that file too. (check comment 0 for an easier to read list)
Comment on attachment 134924 [details] [diff] [review]
patch to remove the  files

ah right you are.
Attachment #134924 - Flags: superreview?(mscott) → superreview+
Attachment #134924 - Flags: review?(timeless) → review+
I don't have time to work on this. I don't even know the current status of the files...
-> default owner
Assignee: mvl → mscott
Assignee: mscott → nobody
The files in comment 0 no longer exist, since they were removed as part of bug 339106. Marking as dupe of that.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 339106
You need to log in before you can comment on or make changes to this bug.