Closed Bug 407494 Opened 18 years ago Closed 18 years ago

Minor dictionary tweaks

Categories

(Core :: Spelling checker, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.9beta3

People

(Reporter: ryanvm, Assigned: ryanvm)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Dictionary tweaks (obsolete) — Splinter Review
Here are some minor tweaks to the recently-added Mozilla words for the Hunspell dictionary. I tweaked some of the word flags to remove redundant entries, which in turn will save a little memory since it'll reduce the size of the dictionary which gets loaded into memory on startup.
Attachment #292225 - Flags: review?(mscott)
After discussing it with Kevin, both of us agree that "uninstall's" is a word which should not be suggested by the spellchecker. v2 adjusts the flags accordingly on uninstall to prevent that suggestion.
Attachment #292225 - Attachment is obsolete: true
Attachment #292243 - Flags: review?(mscott)
Attachment #292225 - Flags: review?(mscott)
Comment on attachment 292243 [details] [diff] [review] Dictionary tweaks v2 Looks good Ryan :)
Attachment #292243 - Flags: review?(mscott) → review+
Comment on attachment 292243 [details] [diff] [review] Dictionary tweaks v2 Requesting approval for post-M10 check-in. This is a super-trivial and safe patch which will improve our spellchecking dictionary a bit.
Attachment #292243 - Flags: approval1.9?
Attachment #292243 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in extensions/spellcheck/locales/en-US/hunspell/en-US.dic; /cvsroot/mozilla/extensions/spellcheck/locales/en-US/hunspell/en-US.dic,v <-- en-US.dic new revision: 1.4; previous revision: 1.3 done Checking in extensions/spellcheck/locales/en-US/hunspell/mozilla_words.diff; /cvsroot/mozilla/extensions/spellcheck/locales/en-US/hunspell/mozilla_words.diff,v <-- mozilla_words.diff new revision: 1.3; previous revision: 1.2 done
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M11
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: