224623 - Remove obsolete files from extensions/spellcheck

Status: RESOLVED DUPLICATE of bug 339106

(Core :: Spelling checker, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

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.

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: patch to remove the files

Comment 2

[Mike Kaply [:mkaply]]

Are we sure that rods will never finish this work?

Comment 3

[[e-mail]]

if he ever wants to, he need but look in bonsai :)

Comment 4

[Scott MacGregor]

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?

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

The patch removes that file too. (check comment 0 for an easier to read list)

Comment 6

[Scott MacGregor]

Comment on attachment 134924 [details] [diff] [review]
patch to remove the  files

ah right you are.

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

I don't have time to work on this. I don't even know the current status of the files...
-> default owner

Comment 8

[Ed Morley [:emorley]]

The files in comment 0 no longer exist, since they were removed as part of bug 339106. Marking as dupe of that.