external/mozc.git
6 hours agoShow the warning message when and only when keymaps for DirectInput mode are changed master
yukawa@google.com [Sat, 25 Oct 2014 06:36:30 +0000 (06:36 +0000)]
Show the warning message when and only when keymaps for DirectInput mode are changed

With this CL, keymap editor shows the warning message when and only when keymaps for DirectInput mode are changed.  Actually this is how imm32-mozc, tsf-mozc, and ibus-mozc behave.

Closes Issue 251.

BUG= Issue mozc:251
TEST=manually done on Ubuntu 14.04

git-svn-id: http://mozc.googlecode.com/svn/trunk@356 a6090854-d499-a067-5803-1114d4e51264

6 days agoStop using non-standard default constructor of std::istream
yukawa@google.com [Sun, 19 Oct 2014 11:13:42 +0000 (11:13 +0000)]
Stop using non-standard default constructor of std::istream

C++11 spec doesn't guarantee that std::istream has a default constructor.
No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@355 a6090854-d499-a067-5803-1114d4e51264

6 days agoFix style violations in header include guards part 2
yukawa@google.com [Sun, 19 Oct 2014 10:56:54 +0000 (10:56 +0000)]
Fix style violations in header include guards part 2

This is a follow up CL for r332
No behavior change is intended with this CL.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@354 a6090854-d499-a067-5803-1114d4e51264

7 days agoSwitch to Android NDK r10c
yukawa@google.com [Sat, 18 Oct 2014 01:45:04 +0000 (01:45 +0000)]
Switch to Android NDK r10c

With this CL, Android NDK r10c is used to build Mozc binaries for Android.

No behavior change is intended.

BUG=none
TEST=manually done with debug APK

git-svn-id: http://mozc.googlecode.com/svn/trunk@353 a6090854-d499-a067-5803-1114d4e51264

13 days agoUse StringPiece to avoid unnecessary instantiation of std::string
yukawa@google.com [Sat, 11 Oct 2014 19:37:26 +0000 (19:37 +0000)]
Use StringPiece to avoid unnecessary instantiation of std::string

With this CL, Segment::Candidate::functional_key() and Segment::Candidate::functional_value() return StringPiece instead of std::string to avoid unnecessary instantiation of std::string.

This is an internal clean-up.  No behavior change is intended.

Patch from Noriyuki Takahashi.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@350 a6090854-d499-a067-5803-1114d4e51264

13 days agoIntroduce Util::ConcatStrings to reduce string copy
yukawa@google.com [Sat, 11 Oct 2014 19:33:02 +0000 (19:33 +0000)]
Introduce Util::ConcatStrings to reduce string copy

This CL introduces Util::ConcatStrings to reduce string copy when concatenating two strings.

This is an internal clean-up.  No behavior change is intended.

Patch from Noriyuki Takahashi.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@349 a6090854-d499-a067-5803-1114d4e51264

13 days agoDo not make the destructor virtual unless it is necessary part 2
yukawa@google.com [Sat, 11 Oct 2014 19:00:10 +0000 (19:00 +0000)]
Do not make the destructor virtual unless it is necessary part 2

mozc::composer::Segment doesn't need to have virtual destructor since it is not have any other virtual methods right now.

This is an internal clean-up thus no behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@348 a6090854-d499-a067-5803-1114d4e51264

13 days agoDo nothing if the key string passed to the converter is empty
yukawa@google.com [Sat, 11 Oct 2014 18:15:34 +0000 (18:15 +0000)]
Do nothing if the key string passed to the converter is empty

Historically, the Mozc has not accepted an empty conversion key.  It has even treated such a query as a critical error.  Nowadays, however, the Mozc converter should silently ignore such a weird request instead of simply crashing as an assertion failure.

Patch from Hiroyuki Komatsu.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@347 a6090854-d499-a067-5803-1114d4e51264

13 days agoFix Year 2038 Problem in UserHistoryStorage
yukawa@google.com [Sat, 11 Oct 2014 17:39:12 +0000 (17:39 +0000)]
Fix Year 2038 Problem in UserHistoryStorage

This CL fixes Year 2038 Problem only in UserHistoryStorage.  The same problem in other parts should be addressed by someone separately.  Note that it is guaranteed in proto binary format that replacing a uint32 entry with a uint64 item with keeping its field ID doesn't break binary compatibility.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@346 a6090854-d499-a067-5803-1114d4e51264

13 days agoDo not make the destructor virtual unless it is necessary
yukawa@google.com [Sat, 11 Oct 2014 17:31:33 +0000 (17:31 +0000)]
Do not make the destructor virtual unless it is necessary

LRUCache doesn't need to have virtual destructor since it is not have any other virtual method right now.

This is an internal clean-up and no behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@345 a6090854-d499-a067-5803-1114d4e51264

13 days agoAdd supportedOS GUID for Windows 10
yukawa@google.com [Sat, 11 Oct 2014 17:25:23 +0000 (17:25 +0000)]
Add supportedOS GUID for Windows 10

We need to explicitly declare supportedOS GUID for Windows 10 to make GetVersionEx API stop lying.  According to the following article, "8e0f7a12-bfb3-4fe8-b9a5-48fd50a15a9a" is the one for Windows 10.
http://blogs.msdn.com/b/chuckw/archive/2013/09/10/manifest-madness.aspx

No behavior change is expected with this CL, since most of version checking in Mozc for Windows has already been implemented with VerifyVersionInfo API rather than GetVersionEx API.  See comments in SystemUtil::IsPlatformSupported for details.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@344 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoUse apt command instead of apt-get command in Ubuntu 14.04
yukawa@google.com [Sun, 5 Oct 2014 12:11:29 +0000 (12:11 +0000)]
Use apt command instead of apt-get command in Ubuntu 14.04

With this CL, Docker container based on Ubuntu 14.04 is built with apt command instead of apt-get command.

No behavior change is intended.

BUG=none
TEST=compile and unittest in Docker 1.0.1 running on Ubuntu 14.04

git-svn-id: http://mozc.googlecode.com/svn/trunk@343 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoTidy up UserHistoryPredictor::InsertHistory
yukawa@google.com [Sun, 5 Oct 2014 08:25:12 +0000 (08:25 +0000)]
Tidy up UserHistoryPredictor::InsertHistory

This CL does nothing but tidy up UserHistoryPredictor::InsertHistory without changing existing behavior.

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@342 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoReorder logical AND conditions just for readability
yukawa@google.com [Sun, 5 Oct 2014 08:18:24 +0000 (08:18 +0000)]
Reorder logical AND conditions just for readability

This CL does nothing but reorder logical AND conditions to determine then a punctuation should be automatically added into the previously committed segment.

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@341 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoRemove a redundant length check
yukawa@google.com [Sun, 5 Oct 2014 08:07:34 +0000 (08:07 +0000)]
Remove a redundant length check

IsPunctuation(const string &value) returns false when |Util::CharsLen(value) == 1|.  We do not need to check it twice.

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@340 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoRoll jsoncpp 4b687640cbc197e8:11086dd6a7eba042
yukawa@google.com [Sun, 5 Oct 2014 01:01:01 +0000 (01:01 +0000)]
Roll jsoncpp 4b687640cbc197e8:11086dd6a7eba042

This CL rolls jsoncpp commits:
  from: 4b687640cbc197e834ba997a3d310f940009643b
    to: 11086dd6a7eba04289944367ca82cea71299ed70

Full commit summary is available here.
https://github.com/open-source-parsers/jsoncpp/compare/4b687640cbc197e834ba997a3d310f940009643b...11086dd6a7eba04289944367ca82cea71299ed70

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@339 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoRoll gyp r1972:r1987
yukawa@google.com [Sun, 5 Oct 2014 00:54:22 +0000 (00:54 +0000)]
Roll gyp r1972:r1987

This CL rolls gyp r1972:r1987. Full commit summary is available here.
https://code.google.com/p/gyp/source/list?start=1987&num=16

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@338 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoRoll breakpad r1354:r1374
yukawa@google.com [Sun, 5 Oct 2014 00:51:06 +0000 (00:51 +0000)]
Roll breakpad r1354:r1374

This CL rolls breakpad r1354:r1374. Full commit summary is available here.
https://code.google.com/p/google-breakpad/source/list?start=1374&num=21

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@337 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoUse Util::EndsWith when appropriate
yukawa@google.com [Sat, 4 Oct 2014 23:29:52 +0000 (23:29 +0000)]
Use Util::EndsWith when appropriate

This CL replaces a piece of code with Util::EndsWith as it behaves equivalently.  No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@336 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoStop unnecessarily using uint64 variable for uint32 data part 2
yukawa@google.com [Sat, 4 Oct 2014 23:21:46 +0000 (23:21 +0000)]
Stop unnecessarily using uint64 variable for uint32 data part 2

UserHistoryPredictor::LearningSegmentFingerprint returns uint32 data.  We don't need to use set<uint64> there.
No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@335 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoStop unnecessarily using uint64 variable for uint32 data
yukawa@google.com [Sat, 4 Oct 2014 23:03:35 +0000 (23:03 +0000)]
Stop unnecessarily using uint64 variable for uint32 data

UserHistoryPredictor::LearningSegmentFingerprint returns uint32 data.  We don't need to use set<uint64> there.

No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@334 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoFix a typo in a test name
yukawa@google.com [Sat, 4 Oct 2014 20:51:53 +0000 (20:51 +0000)]
Fix a typo in a test name

This is just a coding style fix.  No behavior change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@333 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoFix style violations in header include guards
yukawa@google.com [Sat, 4 Oct 2014 20:20:15 +0000 (20:20 +0000)]
Fix style violations in header include guards

This CL style violations in header include guards.
No behavior change is intended with this CL.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@332 a6090854-d499-a067-5803-1114d4e51264

2 weeks agoPropagate information of partial suggestion in the number rewriter
yukawa@google.com [Sat, 4 Oct 2014 19:47:10 +0000 (19:47 +0000)]
Propagate information of partial suggestion in the number rewriter

The following pieces of information are merged with this CL:
- Candidate::consumed_key_size
- Candidate::PARTIALLY_KEY_CONSUMED flag
- Candidate::description for partial suggestion

Patch from Noriyuki Takahashi.

Closes Issue 250.

BUG=Issue mozc:250
TEST=manually done on Android

git-svn-id: http://mozc.googlecode.com/svn/trunk@331 a6090854-d499-a067-5803-1114d4e51264

3 weeks agoAdd an all-in-one Dockerfile to build Mozc for Android and Linux desktop (Ubuntu...
yukawa@google.com [Sun, 28 Sep 2014 16:50:54 +0000 (16:50 +0000)]
Add an all-in-one Dockerfile to build Mozc for Android and Linux desktop (Ubuntu 14.04 edition)

This CL adds an all-in-one Dockerfile where everything is ready to build Mozc for Android and Linux desktop.
(NaCl build is currently broken due to the deprecation of Pepper 27 SDK, which should be addressed separately)

To set up or update the Docker container:
  . make-buildenv.sh
To enter into the Docker container:
  . enter-buildenv.sh
To set up the Docker container again from scratch:
  . clobber-make-buildenv.sh

This CL is not supposed to introduce any user-visible change.

BUG=none
TEST=compile and unittest in Docker 1.0.1 running on Ubuntu 14.04

git-svn-id: http://mozc.googlecode.com/svn/trunk@330 a6090854-d499-a067-5803-1114d4e51264

3 weeks agoDisable ITfFnSearchCandidateProvider until ITfIntegratableCandidateListUIElement...
yukawa@google.com [Sun, 28 Sep 2014 14:51:56 +0000 (14:51 +0000)]
Disable ITfFnSearchCandidateProvider until ITfIntegratableCandidateListUIElement is implemented

This is a speculative fix of Issue 249.

As noted in the following MSDN article, we should implement both ITfFnSearchCandidateProvider and ITfIntegratableCandidateListUIElement at the same time to enable IME search integration on Windows.
http://msdn.microsoft.com/en-us/library/windows/apps/hh967425.aspx#ime_for_search_integration

With this CL, ITfFnSearchCandidateProvider is disabled until ITfIntegratableCandidateListUIElement is implemented.

Closes Issue 249.

BUG=Issue mozc:249
TEST=manually done on Windows 8.1 update 1

git-svn-id: http://mozc.googlecode.com/svn/trunk@329 a6090854-d499-a067-5803-1114d4e51264

4 weeks agoUse Android NDK r10b in the reference build environment part 2
yukawa@google.com [Sat, 20 Sep 2014 16:16:31 +0000 (16:16 +0000)]
Use Android NDK r10b in the reference build environment part 2

This CL build failure for Android build caused by r327.

BUG=none
TEST=manually done with debug APK

git-svn-id: http://mozc.googlecode.com/svn/trunk@328 a6090854-d499-a067-5803-1114d4e51264

4 weeks agoUse Android NDK r10b in the reference build environment
yukawa@google.com [Sat, 20 Sep 2014 16:12:29 +0000 (16:12 +0000)]
Use Android NDK r10b in the reference build environment

With this CL, Android NDK r10b is used in the reference build environment.

No behavior change is intended.

BUG=none
TEST=manually done with debug APK

git-svn-id: http://mozc.googlecode.com/svn/trunk@327 a6090854-d499-a067-5803-1114d4e51264

7 weeks agoMake sure to use correct 'NM' and 'READELF' in cross build
yukawa@google.com [Sat, 30 Aug 2014 22:31:15 +0000 (22:31 +0000)]
Make sure to use correct 'NM' and 'READELF' in cross build

Starting with GYP r1971, 'NM'/'NM.host' and 'READELF'/'READELF.host' specified in 'make_global_settings' can be recognized by Ninja generator. We should use them in NaCl and Android build.  See the following commit and discussion in code review for details.
https://code.google.com/p/gyp/source/detail?r=1971
https://codereview.chromium.org/164023009

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@326 a6090854-d499-a067-5803-1114d4e51264

7 weeks agoRoll gyp r1958:r1971
yukawa@google.com [Sat, 30 Aug 2014 22:20:56 +0000 (22:20 +0000)]
Roll gyp r1958:r1971

This CL rolls gyp r1958:r1971. Full commit summary is available here.
https://code.google.com/p/gyp/source/list?start=1971&num=14

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@325 a6090854-d499-a067-5803-1114d4e51264

2 months agoUpdate copyright year notice in the about dialog
yukawa@google.com [Sun, 24 Aug 2014 19:39:58 +0000 (19:39 +0000)]
Update copyright year notice in the about dialog

This updates copyright year notice in the about dialog, which should have been included in r185.

This is just an update of a text resource in the about dialog. No functional change is intended accordingly.

BUG=none
TEST=manually done

git-svn-id: http://mozc.googlecode.com/svn/trunk@324 a6090854-d499-a067-5803-1114d4e51264

2 months agoUpdate line number attributes in Qt message files
yukawa@google.com [Sun, 24 Aug 2014 19:36:00 +0000 (19:36 +0000)]
Update line number attributes in Qt message files

This is a follow up for r131, which added and removed some lines in about_dialog.cc.
Basically Qt doesn't use these line numbers for production code.  Therefore no user visible change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@323 a6090854-d499-a067-5803-1114d4e51264

2 months agoFix a typo in a build rule for OS X
yukawa@google.com [Sat, 16 Aug 2014 18:58:52 +0000 (18:58 +0000)]
Fix a typo in a build rule for OS X

This is a groundwork to enable Ninja for OS X build.  Unlike make, ninja strictly verifies the dependency for build actions.  Typos in the build rules simply result in build errors.

No behavioral change is intended.

BUG=Issue mozc:247
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@318 a6090854-d499-a067-5803-1114d4e51264

2 months agoIntroduce a new keymap for Chromium OS device
yukawa@google.com [Sat, 16 Aug 2014 17:48:03 +0000 (17:48 +0000)]
Introduce a new keymap for Chromium OS device

This CL introduces a new keymap replace the default keymap for Chromium OS.

No behavioral change is intended except for other platforms.  This keymap should be visible in and only in Chromium OS at the moment.

Patch from Hiroshi Sumita.

BUG=Issue chromium:310698
TEST=Manually done

git-svn-id: http://mozc.googlecode.com/svn/trunk@317 a6090854-d499-a067-5803-1114d4e51264

2 months agoMove GetDefaultKeyMap from keymap::KeyMapManager to config::ConfigHandler
yukawa@google.com [Sat, 16 Aug 2014 17:26:12 +0000 (17:26 +0000)]
Move GetDefaultKeyMap from keymap::KeyMapManager to config::ConfigHandler

This CL moves GetDefaultKeyMap method from keymap::KeyMapManager to config::ConfigHandler as a refactoring.
No behavioral change is intended.

Patch from Hiroshi Sumita.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@316 a6090854-d499-a067-5803-1114d4e51264

2 months agoSupport 'Set input mode to X' in DirectInput mode on Linux, NaCl and Android
yukawa@google.com [Sat, 16 Aug 2014 16:36:44 +0000 (16:36 +0000)]
Support 'Set input mode to X' in DirectInput mode on Linux, NaCl and Android

On Windows, following keymap commands have already been be functional even when assigned to DirectInput mode.
- Set input mode to Hiragana
- Set input mode to full-width alphanumeric
- Set input mode to full-width katakana
- Set input mode to half-width alphanumeric
- Set input mode to half-width katakana

With this CL, these commands will be functional on Linux, NaCl and Android.

No behavioral change is intended for Windows and OS X.

Closes Issue 246.

BUG=Issue mozc:246
TEST=Manually done on Ubuntu 14.04

git-svn-id: http://mozc.googlecode.com/svn/trunk@315 a6090854-d499-a067-5803-1114d4e51264

2 months agoEnable Ninja's console pool feature for 'ant' tasks in Android build
yukawa@google.com [Sat, 16 Aug 2014 14:48:30 +0000 (14:48 +0000)]
Enable Ninja's console pool feature for 'ant' tasks in Android build

With this CL, stdout messages from 'ant' command are immediately displayed in the console, without waiting for the task completion.

Note that the console pool is available in Ninja 1.5 and later.
http://martine.github.io/ninja/manual.html#ref_pool

GYP supports Ninja's console pool in r1955 and later.
https://code.google.com/p/gyp/source/detail?r=1955

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@314 a6090854-d499-a067-5803-1114d4e51264

2 months agoSuppress build time message for the clean build log, especially when being built...
yukawa@google.com [Sat, 16 Aug 2014 13:56:26 +0000 (13:56 +0000)]
Suppress build time message for the clean build log, especially when being built with Ninja

This CL makes the build log a bit less verbose to keep build log clean and tidy, especially when being built with Ninja.

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@313 a6090854-d499-a067-5803-1114d4e51264

2 months agoUpdate symbol dictionary
yukawa@google.com [Sun, 10 Aug 2014 05:59:49 +0000 (05:59 +0000)]
Update symbol dictionary

Following entries are updated to have further readings.
- U+2025: TWO DOT LEADER
- U+2026: HORIZONTAL ELLIPSIS
- U+2295: CIRCLED PLUS

Following entries are newly introduced.
- U+FE19: PRESENTATION FORM FOR VERTICAL HORIZONTAL ELLIPSIS
- U+FE30: PRESENTATION FORM FOR VERTICAL TWO DOT LEADER

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@312 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll protobuf r489:r512
yukawa@google.com [Sat, 9 Aug 2014 19:55:18 +0000 (19:55 +0000)]
Roll protobuf r489:r512

This CL rolls protobuf r489:r512. Full commit summary is available here.
https://code.google.com/p/protobuf/source/list?start=512&num=24

Visual C++ 2013 requires this revision.  See Issue protobuf:531 for details.

No behavioral change is intended.

Closes Issue 244.

BUG=Issue mozc:244
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@311 a6090854-d499-a067-5803-1114d4e51264

2 months agoInclude Visual C++ 2013 merge modules when necessary
yukawa@google.com [Sat, 9 Aug 2014 19:47:50 +0000 (19:47 +0000)]
Include Visual C++ 2013 merge modules when necessary

With this CL, merge modules of Visual C++ 2013 are included into the installer when necessary.

BUG=Issue mozc:244
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@310 a6090854-d499-a067-5803-1114d4e51264

2 months agoUse the workaroud for KB 813540 only for Visual C++ 2010
yukawa@google.com [Sat, 9 Aug 2014 19:43:32 +0000 (19:43 +0000)]
Use the workaroud for KB 813540 only for Visual C++ 2010

Visual C++ 2012 and later contain the fix of KB 813540.
No behavioral change is intended.

BUG=Issue mozc:244
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@309 a6090854-d499-a067-5803-1114d4e51264

2 months agoRemove an unnesesary blank line
yukawa@google.com [Sat, 9 Aug 2014 19:17:03 +0000 (19:17 +0000)]
Remove an unnesesary blank line

This CL is just a style fix.

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@308 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll WTL r460:r587 part 2
yukawa@google.com [Sun, 3 Aug 2014 11:02:25 +0000 (11:02 +0000)]
Roll WTL r460:r587 part 2

This CL fixes checkout failure on Windows caused by r291 due to case-sensitive URL.

No behavioral change is intended.

BUG=none
TEST=gclient and unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@307 a6090854-d499-a067-5803-1114d4e51264

2 months agoRemove an unnecessary directry search rule
yukawa@google.com [Sun, 3 Aug 2014 08:54:22 +0000 (08:54 +0000)]
Remove an unnecessary directry search rule

'mozc_build_tools/mac' was used for 2-pass build.  It should have been removed in r214.

No behavioral change is intended.

BUG=Issue mozc:223
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@303 a6090854-d499-a067-5803-1114d4e51264

2 months agoSwitch to libc++ for better C++11 support
yukawa@google.com [Sun, 3 Aug 2014 08:49:36 +0000 (08:49 +0000)]
Switch to libc++ for better C++11 support

With this CL, Mozc on OS X is built with libc++ instead of legacy libstdc++.  This enables us to use C++11 libraries such as std::unique_ptr in Mac binaries.

No behavioral change is intended.

BUG=Issue mozc:219
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@302 a6090854-d499-a067-5803-1114d4e51264

2 months agoChange the minimum supported OS X version to 10.7
yukawa@google.com [Sun, 3 Aug 2014 08:28:48 +0000 (08:28 +0000)]
Change the minimum supported OS X version to 10.7

With this CL, the default minimum supported OS X version is changed to 10.7.  Hereafter we no longer take care of OS X 10.5/10.6 in Mozc project.

Closes Issue 242

BUG=Issue mozc:242
TEST=manually done

git-svn-id: http://mozc.googlecode.com/svn/trunk@301 a6090854-d499-a067-5803-1114d4e51264

2 months agoRely on predictable Mach port name for OS X
yukawa@google.com [Sun, 3 Aug 2014 07:55:25 +0000 (07:55 +0000)]
Rely on predictable Mach port name for OS X

With this CL, Mozc no longer enumerates all the job objects simply to find relevant Mach port name.  In fact, this was not necessary because our port names are predictable.

Closes Issue 241

Patch from Hiroyuki Komatsu.

BUG=Issue mozc:241
TEST=manually done on OS X 10.10 beta

git-svn-id: http://mozc.googlecode.com/svn/trunk@300 a6090854-d499-a067-5803-1114d4e51264

2 months agoChange the group ID of the installed files from admin(80) to wheel(0) for Mac OS X
yukawa@google.com [Sun, 3 Aug 2014 07:05:52 +0000 (07:05 +0000)]
Change the group ID of the installed files from admin(80) to wheel(0) for Mac OS X

Based on discussion with Mac experts.

BUG=none
TEST=manually done

git-svn-id: http://mozc.googlecode.com/svn/trunk@299 a6090854-d499-a067-5803-1114d4e51264

2 months agoFix missing copyright notice and license notice at the top of the file
yukawa@google.com [Sun, 3 Aug 2014 06:44:27 +0000 (06:44 +0000)]
Fix missing copyright notice and license notice at the top of the file

Somehow these files lacked copyright notice and license notice at the top of the file.  This CL addresses the issue.

Neither behavioral nor licensing change is intended with this CL.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@298 a6090854-d499-a067-5803-1114d4e51264

2 months agoSimplify build rules for Mac
yukawa@google.com [Sun, 3 Aug 2014 06:33:46 +0000 (06:33 +0000)]
Simplify build rules for Mac

This CL simplifies build rules for Mac by renaming remaining *.m with *.mm.

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@297 a6090854-d499-a067-5803-1114d4e51264

2 months agoUse Android NDK r10
yukawa@google.com [Sat, 2 Aug 2014 18:07:35 +0000 (18:07 +0000)]
Use Android NDK r10

With this CL, Android NDK version is bumped from r9d to r10 in the reference build environment.

No behavioral change is intended.

BUG=none
TEST=manually done with debug APK

git-svn-id: http://mozc.googlecode.com/svn/trunk@296 a6090854-d499-a067-5803-1114d4e51264

2 months agoRemove unused function in auto-generated code
yukawa@google.com [Sat, 2 Aug 2014 17:19:09 +0000 (17:19 +0000)]
Remove unused function in auto-generated code

This CL removes an unused function that was automatically generated by Lemon Parser.

No behavioral change is intended.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@295 a6090854-d499-a067-5803-1114d4e51264

2 months agoFix code signing error on Mac OS X Marveriks
yukawa@google.com [Sat, 2 Aug 2014 17:01:37 +0000 (17:01 +0000)]
Fix code signing error on Mac OS X Marveriks

This CL addresses several issues in code signing on Mac OS X Marveriks and later.
- codesign command should be used with '--deep' option.
- Info.plist should be copied under Resources/

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@294 a6090854-d499-a067-5803-1114d4e51264

2 months agoRemove a workaround against copy-on-write implementation of std::string
yukawa@google.com [Sat, 2 Aug 2014 16:47:41 +0000 (16:47 +0000)]
Remove a workaround against copy-on-write implementation of std::string

Copy-on-write implementation of std::string is not allowed in C++11.  This CL simply removes the workaround against it except for NaCl where we haven't yet confirmed that the std::string implementation C++11 spec.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@293 a6090854-d499-a067-5803-1114d4e51264

2 months agoUse C++11 auto keyword
yukawa@google.com [Sat, 2 Aug 2014 16:23:43 +0000 (16:23 +0000)]
Use C++11 auto keyword

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@292 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll WTL r460:r587
yukawa@google.com [Sat, 2 Aug 2014 15:53:15 +0000 (15:53 +0000)]
Roll WTL r460:r587

This CL rolls WTL r460:r587.
http://sourceforge.net/p/wtl/code/587/log/
http://sourceforge.net/p/wtl/code/562/log/
http://sourceforge.net/p/wtl/code/537/log/
http://sourceforge.net/p/wtl/code/512/log/
http://sourceforge.net/p/wtl/code/487/log/
http://sourceforge.net/p/wtl/code/462/log/

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@291 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll protobuf r463:r488
yukawa@google.com [Sat, 2 Aug 2014 15:43:14 +0000 (15:43 +0000)]
Roll protobuf r463:r488

This CL rolls protobuf r463:r488. Full commit summary is available here.
https://code.google.com/p/protobuf/source/list?start=488&num=27

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@290 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll jsoncpp ea0797351fbabd3e:3a0c4fcc82d25d18
yukawa@google.com [Sat, 2 Aug 2014 15:39:53 +0000 (15:39 +0000)]
Roll jsoncpp ea0797351fbabd3e:3a0c4fcc82d25d18

This CL rolls jsoncpp ea0797351fbabd3ec9dc98304c7c12c6cd1c61dd:3a0c4fcc82d25d189b8107e07462effbab9f8e1b. Full commit summary is available here.
https://github.com/open-source-parsers/jsoncpp/compare/ea0797351fbabd3ec9dc98304c7c12c6cd1c61dd...3a0c4fcc82d25d189b8107e07462effbab9f8e1b

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@289 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll gyp r1949:r1957
yukawa@google.com [Sat, 2 Aug 2014 15:30:47 +0000 (15:30 +0000)]
Roll gyp r1949:r1957

This CL rolls gyp r1949:r1957. Full commit summary is available here.
https://code.google.com/p/gyp/source/list?start=1957&num=9

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@288 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll gtest r682:r692 and gmock r472:r485
yukawa@google.com [Sat, 2 Aug 2014 15:28:13 +0000 (15:28 +0000)]
Roll gtest r682:r692 and gmock r472:r485

This CL rolls gtest r682:r692 and gmock r472:r485. Full commit summary is available here.
https://code.google.com/p/googletest/source/list?start=692&num=11
https://code.google.com/p/googlemock/source/list?start=485&num=14

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@287 a6090854-d499-a067-5803-1114d4e51264

2 months agoRoll breakpad r1239:r1353
yukawa@google.com [Sat, 2 Aug 2014 15:20:38 +0000 (15:20 +0000)]
Roll breakpad r1239:r1353

This CL rolls breakpad r1239:r1353. Full commit summary is available here.
https://code.google.com/p/google-breakpad/source/list?num=115&start=1353

No behavioral change is intended.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@286 a6090854-d499-a067-5803-1114d4e51264

2 months agoBetter conforms to the emacs library standard
yukawa@google.com [Sat, 2 Aug 2014 07:56:15 +0000 (07:56 +0000)]
Better conforms to the emacs library standard

See the following links for details.
https://www.gnu.org/software/emacs/manual/html_node/elisp/Simple-Packages.html
https://www.gnu.org/software/emacs/manual/html_node/elisp/Library-Headers.html

Closes Issue 213.

Patch from Yuki Shiino.

BUG=Issue mozc:213
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@285 a6090854-d499-a067-5803-1114d4e51264

2 months agoAdds autoload magic comments
yukawa@google.com [Sat, 2 Aug 2014 07:50:24 +0000 (07:50 +0000)]
Adds autoload magic comments

Makes use of some symbols automatically load the mozc.el.

See the following links for details.
https://www.gnu.org/software/emacs/manual/html_node/elisp/Packaging-Basics.html
https://www.gnu.org/software/emacs/manual/html_node/elisp/Autoload.html

Patch from Yuki Shiino.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@284 a6090854-d499-a067-5803-1114d4e51264

2 months agoUse more appropriate and descriptive constant for OK/Cancel dialog
yukawa@google.com [Sun, 27 Jul 2014 17:26:14 +0000 (17:26 +0000)]
Use more appropriate and descriptive constant for OK/Cancel dialog

Both android.R.string.yes and android.R.string.ok are mapped into "OK" at least in in English resource.  If OK/Cancel dialog is OK, we should use android.R.string.ok.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@283 a6090854-d499-a067-5803-1114d4e51264

3 months agoInclude twelvekeys_toggle_flick_alphabet_scenario.txt in the test scenario list
yukawa@google.com [Sun, 20 Jul 2014 14:14:55 +0000 (14:14 +0000)]
Include twelvekeys_toggle_flick_alphabet_scenario.txt in the test scenario list

Somehow twelvekeys_toggle_flick_alphabet_scenario.txt has not been included in the the test scenario list.  With this CL, that test is included in the test scenario list, as it should be.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@282 a6090854-d499-a067-5803-1114d4e51264

3 months agoAdd an all-in-one Dockerfile to build Mozc for Android, NaCl, and Linux desktop ...
yukawa@google.com [Sat, 12 Jul 2014 18:30:50 +0000 (18:30 +0000)]
Add an all-in-one Dockerfile to build Mozc for Android, NaCl, and Linux desktop (Ubuntu 12.04 edition)

This CL adds an all-in-one Dockerfile where everything is ready to build Mozc for Android, NaCl, and Linux desktop.

To set up or update the Docker container:
  . make-buildenv.sh
To enter into the Docker container:
  . enter-buildenv.sh
To set up the Docker container again from scratch:
  . clobber-make-buildenv.sh

This CL is not supposed to introduce any user-visible change.

Closes Issue 240

BUG= Issue mozc:240
TEST=compile and unittest in Docker 1.0 running on Ubuntu 14.04

git-svn-id: http://mozc.googlecode.com/svn/trunk@271 a6090854-d499-a067-5803-1114d4e51264

3 months agoMinor clean up in gyp file
yukawa@google.com [Sat, 12 Jul 2014 15:31:59 +0000 (15:31 +0000)]
Minor clean up in gyp file

As a general rule in Mozc, 'genproto_*' gyp target is not supposed to be used directly from other *.gyp files.  Actually it doesn't work well for cross compile scenario.  Corresponding '*_protocol' target should be used instead.

This CL is not supposed to introduce any user-visivle behavior change.

BUG=Issue mozc:222
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@270 a6090854-d499-a067-5803-1114d4e51264

3 months agoZero query suggestion shouldn't be triggered when committing with key event
yukawa@google.com [Sat, 12 Jul 2014 14:59:25 +0000 (14:59 +0000)]
Zero query suggestion shouldn't be triggered when committing with key event

It turned out that our zero query suggestion is implemented in an ad-hoc way, by naively and recklessly calling Session::Suggest from Session::Commit. This is actually dangerous because most of callers of Session::Commit never expect that Session::Commit could be called Session::Suggest internally as a side effect.  This CL addresses such situation.

Anyway, oru current implementation of zero query suggestion is considered to be bug-prone and some refactoring might be required.

Closes Issue 239.

BUG= Issue mozc:239
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@269 a6090854-d499-a067-5803-1114d4e51264

3 months agoUse Ninja to build Android, NaCl, and Linux Desktop binaries part 3
yukawa@google.com [Sat, 12 Jul 2014 07:43:37 +0000 (07:43 +0000)]
Use Ninja to build Android, NaCl, and Linux Desktop binaries part 3

This CL fixes runtime crash on Android.

'product_dir' feature in GYP seems to be interpreted differently between Make generator and Ninja generator especially regarding relative path resolution.  This difference should have been taken care of in r266.  Otherwise, the APK lacks libmozc.so and Mozc for Android crashes whenever it is being launched.

With this CL, 'product_dir' used to build libmozc.so is specified with an absolute path rather than relative path so that Mozc for Android can be built correctly.

BUG= Issue mozc:222
TEST=manually tested on Android

git-svn-id: http://mozc.googlecode.com/svn/trunk@268 a6090854-d499-a067-5803-1114d4e51264

3 months agoUse Ninja to build Android, NaCl, and Linux Desktop binaries part 2
yukawa@google.com [Sat, 12 Jul 2014 07:17:21 +0000 (07:17 +0000)]
Use Ninja to build Android, NaCl, and Linux Desktop binaries part 2

This CL fixes gyp error on Windows.

GYP_CROSSCOMPILE is not yet ready to be enabled on Windows.  It shouldn't have been enabled in r266.

No user-visivle behavior is supposed to be changed with this CL.

BUG=Issue mozc:222
TEST=unittest on Windows

git-svn-id: http://mozc.googlecode.com/svn/trunk@267 a6090854-d499-a067-5803-1114d4e51264

3 months agoUse Ninja to build Android, NaCl, and Linux Desktop binaries
yukawa@google.com [Sun, 6 Jul 2014 07:49:48 +0000 (07:49 +0000)]
Use Ninja to build Android, NaCl, and Linux Desktop binaries

With this CL, Mozc starts using Ninja instead of Make to build Android, NaCl, and Linux Desktop binaries.  Note that Windows binaries have already been built with Ninja for one and a half years.

This CL is supposed not to change any user-visible behavior.

Closes Issue 222.

BUG=Issue mozc:222
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@266 a6090854-d499-a067-5803-1114d4e51264

3 months agoPull GYP r1948
yukawa@google.com [Sun, 6 Jul 2014 07:35:34 +0000 (07:35 +0000)]
Pull GYP r1948

This CL pulls recent changes in GYP upstream so as to fix incompatibility between Make generator and Ninja generator when cross-compile support is enabled in GYP.  See Issue gyp:434 for details.

This CL is supposed not to change any user-visible behavior.

BUG=mozc:222
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@265 a6090854-d499-a067-5803-1114d4e51264

3 months agoSpecify the absolute path of the project root directory for include directories
yukawa@google.com [Sat, 5 Jul 2014 16:26:27 +0000 (16:26 +0000)]
Specify the absolute path of the project root directory for include directories

This is a groundwork for Ninja migration on Linux.  No user-visible behavioral changes are expected with this CL.

BUG=mozc:222
TEST=compile and unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@253 a6090854-d499-a067-5803-1114d4e51264

3 months agoDisable ITfFnGetLinguisticAlternates until the root cause of Explorer crashes is...
yukawa@google.com [Sat, 5 Jul 2014 08:37:30 +0000 (08:37 +0000)]
Disable ITfFnGetLinguisticAlternates until the root cause of Explorer crashes is addressed

With this CL, ITfFnGetLinguisticAlternates support is tentatively disabled due to the crash of Start Screen on Windows 8.1 update1.  Perhaps Explorer requires an IME to implement not only ITfFnGetLinguisticAlternates but also ITfIntegratableCandidateListUIElement for the Start Screen.  Anyway, further investigation seems to be required before enabling ITfFnGetLinguisticAlternates again.

Closes Issue 236.

BUG=mozc:236
TEST=manually done on Windows 8.1 update1

git-svn-id: http://mozc.googlecode.com/svn/trunk@252 a6090854-d499-a067-5803-1114d4e51264

3 months agoSpecify text label for the menu item of MozcTool in ibus-mozc
yukawa@google.com [Sat, 5 Jul 2014 02:52:20 +0000 (02:52 +0000)]
Specify text label for the menu item of MozcTool in ibus-mozc

Gnome Shell prefers to use text label over icon. Without the text label, a blank item is displayed.

Closes Issue 233.

BUG=mozc:233
TEST=manually done on Fedora 20

git-svn-id: http://mozc.googlecode.com/svn/trunk@251 a6090854-d499-a067-5803-1114d4e51264

3 months agoDo not pass incomplete types to STL containers
yukawa@google.com [Sat, 28 Jun 2014 17:24:06 +0000 (17:24 +0000)]
Do not pass incomplete types to STL containers

Using STL containers with incomplete types are fragile in practice, and easily results in an undefined behavior.  On the other hand, including generated header files in a public header file is also considered harmful, because it easily results in non-deterministic compile errors unless hidden build dependencies are carefully carefully annotated in GYP file.  Thus this CL isn't great as it should be because it deliberately does the later to resolve the former.  Hopefully in future we will revisit here to stop including generated header files in a public header file.

This is just a build-specific issue thus no user-visible changes are expected.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@250 a6090854-d499-a067-5803-1114d4e51264

3 months agoExplicitly include <unistd.h> for OSX
yukawa@google.com [Sat, 28 Jun 2014 15:59:49 +0000 (15:59 +0000)]
Explicitly include <unistd.h> for OSX

geteuid()/getuid() require <unistd.h>.  This is just a build issue thus no user-visible changes are expected.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@249 a6090854-d499-a067-5803-1114d4e51264

3 months agoStop using template to implement BasicCodeGenByteArrayStreamBuf
yukawa@google.com [Sat, 28 Jun 2014 15:48:46 +0000 (15:48 +0000)]
Stop using template to implement BasicCodeGenByteArrayStreamBuf

Implementing BasicCodeGenByteArrayStreamBuf as a template class might have been be overkill because no one has used it with non-char type (e.g. wchar).  This CL simplifies the implementation of CodeGenByteArrayStream by assuming T=char everywhere.

No user-visible changes are expected with this CL.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@248 a6090854-d499-a067-5803-1114d4e51264

3 months agoExplicity invoke bash for make-standalone-toolchain.sh
yukawa@google.com [Sat, 28 Jun 2014 15:12:40 +0000 (15:12 +0000)]
Explicity invoke bash for make-standalone-toolchain.sh

Unfortunately make-standalone-toolchain.sh lacks shebang as of NDK r9d so it would be better to call some shell explicitly.

No user-visible changes are expected with this CL.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@247 a6090854-d499-a067-5803-1114d4e51264

3 months agoMake config_file_stream_test.cc more robust
yukawa@google.com [Sat, 28 Jun 2014 14:56:09 +0000 (14:56 +0000)]
Make config_file_stream_test.cc more robust

Seems that on some enviroment (e.g. OS X 10.8 w/ Xcode 4.5) peek() does not flip eofbit.  So we also get() to test if the given stream reached to EOF or not.

This CL does not any production code thus no user-visible changes are expected.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@246 a6090854-d499-a067-5803-1114d4e51264

3 months agoReplace NULL with nullptr in mozc::NumberUtil
yukawa@google.com [Sat, 28 Jun 2014 14:10:59 +0000 (14:10 +0000)]
Replace NULL with nullptr in mozc::NumberUtil

This is a mechanical replacement of NULL with nullptr in favor of C++11.  No behavioral changes are expected.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@245 a6090854-d499-a067-5803-1114d4e51264

3 months agoEnable Unicode 6 Emoji conversion by default for desktop (part 4)
yukawa@google.com [Sat, 28 Jun 2014 13:06:57 +0000 (13:06 +0000)]
Enable Unicode 6 Emoji conversion by default for desktop (part 4)

This is the 4th part (hopefully the final part) of an attempt to enable Unicode 6 Emoji conversion by default for desktop.

This CL fixes a behavioral mismatch between ConfigHandlerImpl::SetConfigInternal and ConfigHandler::GetDefaultConfig to address test failure of ConfigHandlerTest.SetConfig on Linux.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@244 a6090854-d499-a067-5803-1114d4e51264

3 months agoUnify nested preprocessor conditions into one line (part 3)
yukawa@google.com [Sat, 28 Jun 2014 12:36:11 +0000 (12:36 +0000)]
Unify nested preprocessor conditions into one line (part 3)

This is a minor style fix to follow up r228 and r241. No functional changes are expected.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@243 a6090854-d499-a067-5803-1114d4e51264

3 months agoMinor style fixes
yukawa@google.com [Sat, 28 Jun 2014 12:29:13 +0000 (12:29 +0000)]
Minor style fixes

Just style fixes.  No behavioral changes are expected with this CL.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@242 a6090854-d499-a067-5803-1114d4e51264

3 months agoUnify nested preprocessor conditions into one line (part 2)
yukawa@google.com [Sat, 28 Jun 2014 11:59:11 +0000 (11:59 +0000)]
Unify nested preprocessor conditions into one line (part 2)

This is a minor style fix to follow up r228. No functional changes are expected.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@241 a6090854-d499-a067-5803-1114d4e51264

4 months agoEnsure that the IPC client is configured before it is used in ibus-mozc
yukawa@google.com [Sun, 22 Jun 2014 04:35:53 +0000 (04:35 +0000)]
Ensure that the IPC client is configured before it is used in ibus-mozc

This CL ensures that ClientInterface::set_client_capability is called before the the IPC client is actually used in ibus-mozc.

This is important because Mozc's IPC client library relies on so-called 2-phase construction. The IPC client must be instantiated by ClientFactory::NewClient(), then should be configured appropriately before it is actually used.

Closes Issue 227.

BUG=mozc:227
TEST=manually done on Linux

git-svn-id: http://mozc.googlecode.com/svn/trunk@234 a6090854-d499-a067-5803-1114d4e51264

4 months agoTeach ibus-mozc about the correct substring position when extracting surrounding...
yukawa@google.com [Sat, 21 Jun 2014 14:20:14 +0000 (14:20 +0000)]
Teach ibus-mozc about the correct substring position when extracting surrounding text (part 2)

This CL closes mozc:226.  With this CL, ibus-mozc starts extracting preceding text and following text correctly.

This CL also addresses a potential crash scenario when ibus_engine_get_surrounding_text returns out-of-range |cursor_pos| and/or |anchor_pos| for some reasons.

BUG=mozc:226
TEST=manually done on Linux

git-svn-id: http://mozc.googlecode.com/svn/trunk@233 a6090854-d499-a067-5803-1114d4e51264

4 months agoTeach ibus-mozc about the correct substring position when extracting surrounding...
yukawa@google.com [Sat, 21 Jun 2014 14:07:15 +0000 (14:07 +0000)]
Teach ibus-mozc about the correct substring position when extracting surrounding text (part 1)

This CL introduces an overload of Util::SubStringPiece that extracts a substring to the end, which will be used in a subsequent CL to fix a bug that ibus-mozc wrongly extracts surrounding text. This is a groundwrok. No functional changes are expected.

BUG=mozc:226
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@232 a6090854-d499-a067-5803-1114d4e51264

4 months agoEnable Unicode 6 Emoji conversion by default for desktop (part 3)
yukawa@google.com [Sat, 21 Jun 2014 12:26:21 +0000 (12:26 +0000)]
Enable Unicode 6 Emoji conversion by default for desktop (part 3)

This is the 3rd part (hopefully the final part) of an attempt to enable Unicode 6 Emoji conversion by default for desktop.

This CL fixes a typo introduced in r230, which results in a compile failure.

BUG=none
TEST=compile

git-svn-id: http://mozc.googlecode.com/svn/trunk@231 a6090854-d499-a067-5803-1114d4e51264

4 months agoEnable Unicode 6 Emoji conversion by default for desktop (part 2)
yukawa@google.com [Sat, 21 Jun 2014 11:56:48 +0000 (11:56 +0000)]
Enable Unicode 6 Emoji conversion by default for desktop (part 2)

This is a follow up CL for r229, which aimed to enable Unicode 6 Emoji conversion by default for desktop, but unintentionally disabled it instead, except for Windows.

With this CL, Unicode 6 Emoji conversion is enabled by default for desktop versions except for Windows 7 and prior versions as originally intended.

BUG=none
TEST=manually done on Linux and Windows

git-svn-id: http://mozc.googlecode.com/svn/trunk@230 a6090854-d499-a067-5803-1114d4e51264

4 months agoEnable Unicode 6 Emoji conversion by default for desktop
yukawa@google.com [Sat, 21 Jun 2014 10:48:45 +0000 (10:48 +0000)]
Enable Unicode 6 Emoji conversion by default for desktop

With this CL, Unicode 6 Emoji conversion is enabled by default for desktop versions of Mozc except for Windows 7 and prior versions.

Note that the behavior of Android Mozc is still a bit complicated because of so-called carriers' Emoji. See EmojiCarrierType in session/commands.proto for details.

BUG=none
TEST=manually done

git-svn-id: http://mozc.googlecode.com/svn/trunk@229 a6090854-d499-a067-5803-1114d4e51264

4 months agoUnify nested preprocessor conditions into one line
yukawa@google.com [Sat, 21 Jun 2014 10:32:28 +0000 (10:32 +0000)]
Unify nested preprocessor conditions into one line

This is a minor style fix for nested preprocessor conditions. No functional changes are expected.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@228 a6090854-d499-a067-5803-1114d4e51264

4 months agoStrictly check the range of the operands for integer operations performed in NumberUtil
yukawa@google.com [Sat, 21 Jun 2014 08:49:55 +0000 (08:49 +0000)]
Strictly check the range of the operands for integer operations performed in NumberUtil

Previously, NumberUtil have checked the result of some integer operations instead of checking the range of operands. This isn't good idea, especially for the really trick cases such as |-INT_MIN|.

This CL enforces strict range check of the operands rather than the result for integer operations performed in NumberUtil.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@227 a6090854-d499-a067-5803-1114d4e51264

4 months agoSimplify test cases for SafeStrToDouble/SafeStrToFloat
yukawa@google.com [Sat, 21 Jun 2014 07:58:31 +0000 (07:58 +0000)]
Simplify test cases for SafeStrToDouble/SafeStrToFloat

This CL simplifies test cases for SafeStrToDouble/SafeStrToFloat so as not to assert that SafeStrToFloat/SafeStrToDouble always fail to parse a hexadecimal number with Visual C++ and NDK. Such assertion is too fragile for future update of toolchains and/or platform libraries.

This CL never changes production code.  No functional changes are expected.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@226 a6090854-d499-a067-5803-1114d4e51264

4 months agoRemove a redundant constant definition
yukawa@google.com [Sat, 21 Jun 2014 07:36:11 +0000 (07:36 +0000)]
Remove a redundant constant definition

kint64min has already been debined in port.h. We don't need to have another constant in number_util.cc locally.

No functional changes are expected with this CL.

BUG=none
TEST=compile and unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@225 a6090854-d499-a067-5803-1114d4e51264

4 months agoUtilTest::StripWhiteSpaces should work with a single letter text.
yukawa@google.com [Sat, 14 Jun 2014 22:49:15 +0000 (22:49 +0000)]
UtilTest::StripWhiteSpaces should work with a single letter text.

This CL fixes a bug that UtilTest::StripWhiteSpaces couldn't handle a single letter text such as "a".  Though it is not sure if this was observable as a user-visible issue.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@224 a6090854-d499-a067-5803-1114d4e51264

4 months agoUse different internal namespace for gflags
yukawa@google.com [Sat, 14 Jun 2014 22:39:00 +0000 (22:39 +0000)]
Use different internal namespace for gflags

Mozc's flags system, which is implemented in base/flags.h, looks quite similar to OSS gflags library.  This is not surprising, because they originate from the same Google internal library actually.  As a result, however, currently OSS Mozc and OSS gflags are using the same name for internal variables, which results in a potential risk of conflicting symbol name during link time when you mix Mozc's code with other projects that also rely on gflags.

To eliminate this potential risk, this CL renames the internal namespace from fL##shorttype to mozc_flags_fL##shorttype.  No user-visible behavioral changes are expected with this CL.

BUG=none
TEST=unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@223 a6090854-d499-a067-5803-1114d4e51264

4 months agoFix the version check logic of the downloadable dictionary for NaCl-Mozc.
yukawa@google.com [Sat, 14 Jun 2014 22:05:23 +0000 (22:05 +0000)]
Fix the version check logic of the downloadable dictionary for NaCl-Mozc.

This CL fixes a bug in the version check of the downloadable dictionary for NaCl-Mozc.  Note that only NaCl-Mozc built with GOOGLE_JAPANESE_INPUT_BUILD enabled is affected.

BUG=none
TEST=none

git-svn-id: http://mozc.googlecode.com/svn/trunk@222 a6090854-d499-a067-5803-1114d4e51264

4 months agoDo not use shell syntax in gyp (part 2)
yukawa@google.com [Sat, 7 Jun 2014 14:08:04 +0000 (14:08 +0000)]
Do not use shell syntax in gyp (part 2)

This is a groundwork for Ninja migration on Linux. No user-visible behavioral changes are expected with this CL.

'$' is reserved for Ninja internal use. We shouldn't rely on it in gyp layer.

With this CL, a conditional action regarding symbol removal is handled in a python script rather than the gyp (and therefore Ninja) layer.

BUG=mozc:222
TEST=compile and unittest

git-svn-id: http://mozc.googlecode.com/svn/trunk@221 a6090854-d499-a067-5803-1114d4e51264