Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish CLs from internal repository (CL 87592180 - CL 91370456) #333

Merged
merged 10 commits into from
Nov 15, 2015

Conversation

yukawa
Copy link
Collaborator

@yukawa yukawa commented Nov 15, 2015

This series of commits corresponds to CLs in Google internal repository from CL 87592180 to CL 91370456.

Noriyuki Takahashi and others added 10 commits November 14, 2015 09:48
encoding_util is used only by mozc_tool.  Moving it to src/gui/base
consolidates build dependencies and improves code portability.

This is just a refactoring.  No behavior change is intended.

BUG=#252
TEST=unittest
REF_BUG=19010851
REF_CL=87592180
 - Part 1: a110641
 - Part 2: d71135a

BUG=
TEST=unittest
REF_BUG=18587000
REF_CL=88403050
Historically Output::all_candidate_words field has been omitted only in
NaCl mainly because of performance reasons.  With this CL, that field is
enabled even in NaCl build.

To mitigate the impact on performance, this CL also introduces
Request::candidates_size_limit field so that clients can change the
maximum number of candidates for each conversion request.

BUG=
TEST=unittest
REF_BUG=
REF_CL=88403136
This CL fixes #331, by tweaking the rules to apply suggestion filter.
On mobile suggestion filter is no longer applied to exact match results
generated by realtime conversion.

BUG=#331
TEST=unittest
REF_BUG=19832938
REF_CL=89478578
Unlinking-file-then-reload doesn't work here because reload fails
because the file has already gone.  Therefore, we make the storage empty
and save it.

BUG=#332
TEST=unittest
REF_BUG=19940325
REF_CL=90124705
- Add transliterated name for variant type.
- Add comment for each type.
- Fixed some errors.

REF_BUG=19918925
REF_CL=90140131,90388075
This is a preparation to add 3-digit rewriting by parameterizing out
constant '4'.

BUG=
TEST=compile
REF_BUG=20059084
REF_CL=90573575
BUG=
TEST=
REF_BUG=
REF_CL=90781361
This CL updates DateRewriter to rewrite 3-digits segments to date and
time.

3-digits are splitted in two ways (DDD to D+DD and DD+D).  The
candidates are organized in the following order:
  1. description (date/time)
  2. split type (D+DD/DD+D)
  3. string format (XX:XX, etc)

BUG=
TEST=unittest
REF_BUG=20059084
REF_CL=90971460
With this CL, EncodingUtil::SJISToUTF8 no longer depends on any
platform-dependent API, including iconv and Win32 APIs.

Keep in mind that EncodingUtil::SJISToUTF8 is mainly used for importing
user dictionary for MS-IME.  Hence we want EncodingUtil::SJISToUTF8 to
be compatible with the behavior on Windows.

This also closes #27 because we no longer depend on iconv.

BUG=#27,#252
TEST=unittest
REF_BUG=19010851
REF_CL=91370456,107291445
yukawa added a commit that referenced this pull request Nov 15, 2015
Publish CLs from internal repository (CL 87592180 - CL 91370456)
@yukawa yukawa merged commit 95de40f into master Nov 15, 2015
@yukawa yukawa deleted the staging_cl87592180_cl91370456 branch November 15, 2015 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants