Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Issue 966563002: [spell] Roll SCOWL dictionaries for en-US & en-CA. (Closed)

Created:
5 years, 10 months ago by please use gerrit instead
Modified:
5 years, 9 months ago
CC:
chromium-reviews, mwichary, paul.irish
Base URL:
https://chromium.googlesource.com/chromium/deps/hunspell_dictionaries.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

[spell] Roll SCOWL dictionaries for en-US & en-CA. Updated the en-US and en-CA dictionaries to the latest upstream version from SCOWL, but made sure that "alot -> a lot" correction does not disappear, as it is not in SCOWL yet. The updated dictionaries support typographical apostrophes. Because both aff and dic files changed, we incremented major version to 6-0. That's 1 higher than the highest major version of any dictionary. BUG=165079 R=groby@chromium.org TBR=cpu@chromium.org Committed: 294375

Patch Set 1 #

Total comments: 1

Patch Set 2 : Remove OWNERS. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4731 lines, -4347 lines) Patch
M README.chromium View 1 chunk +8 lines, -1 line 0 comments Download
M README_en_CA.txt View 4 chunks +5 lines, -6 lines 0 comments Download
M README_en_US.txt View 4 chunks +5 lines, -6 lines 0 comments Download
A en-CA-6-0.bdic View Binary file 0 comments Download
A en-US-6-0.bdic View Binary file 0 comments Download
M en_CA.aff View 2 chunks +4 lines, -2 lines 0 comments Download
M en_CA.dic View 1378 chunks +2357 lines, -2169 lines 0 comments Download
M en_US.aff View 2 chunks +4 lines, -2 lines 0 comments Download
M en_US.dic View 1368 chunks +2348 lines, -2161 lines 0 comments Download

Messages

Total messages: 14 (7 generated)
please use gerrit instead
Rachel, PTAL.
5 years, 10 months ago (2015-02-27 00:55:48 UTC) #5
groby-ooo-7-16
RS LGTM (change looks correct, but I didn't verify dictionary contents individually) https://codereview.chromium.org/966563002/diff/60001/en_CA.dic File en_CA.dic ...
5 years, 9 months ago (2015-02-27 22:55:31 UTC) #6
please use gerrit instead
On 2015/02/27 22:55:31, groby wrote: > RS LGTM (change looks correct, but I didn't verify ...
5 years, 9 months ago (2015-02-27 23:02:12 UTC) #7
please use gerrit instead
Ben, OWNERS PTAL. Note that I've added an OWNERS file with some of the current ...
5 years, 9 months ago (2015-02-27 23:03:48 UTC) #9
please use gerrit instead
Carlos, OWNERS PTAL. Note that I've added an OWNERS file for third_party/hunspell_dictionaries with some of ...
5 years, 9 months ago (2015-03-06 23:05:05 UTC) #12
please use gerrit instead
Noticed that third_party/OWNERS says to TBR cpu@ or darin@ for changes within third_party. Therefore, I ...
5 years, 9 months ago (2015-03-06 23:39:12 UTC) #13
please use gerrit instead
5 years, 9 months ago (2015-03-06 23:39:48 UTC) #14
Message was sent while issue was closed.
Committed patchset #2 (id:80001) manually as 294375 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698