From owner-svn-ports-head@freebsd.org Wed Jun 3 18:05:33 2020 Return-Path: Delivered-To: svn-ports-head@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 5A6A633D1AF; Wed, 3 Jun 2020 18:05:33 +0000 (UTC) (envelope-from sunpoet@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 49ccJs1bwsz41x7; Wed, 3 Jun 2020 18:05:33 +0000 (UTC) (envelope-from sunpoet@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 2AFC111ED2; Wed, 3 Jun 2020 18:05:32 +0000 (UTC) (envelope-from sunpoet@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id 053I5V3r048570; Wed, 3 Jun 2020 18:05:31 GMT (envelope-from sunpoet@FreeBSD.org) Received: (from sunpoet@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id 053I5Utn048561; Wed, 3 Jun 2020 18:05:30 GMT (envelope-from sunpoet@FreeBSD.org) Message-Id: <202006031805.053I5Utn048561@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: sunpoet set sender to sunpoet@FreeBSD.org using -f From: Sunpoet Po-Chuan Hsieh Date: Wed, 3 Jun 2020 18:05:30 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r537632 - head/japanese/mozc-server/files X-SVN-Group: ports-head X-SVN-Commit-Author: sunpoet X-SVN-Commit-Paths: head/japanese/mozc-server/files X-SVN-Commit-Revision: 537632 X-SVN-Commit-Repository: ports MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.33 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 03 Jun 2020 18:05:33 -0000 Author: sunpoet Date: Wed Jun 3 18:05:29 2020 New Revision: 537632 URL: https://svnweb.freebsd.org/changeset/ports/537632 Log: Fix build with protobuf 3.12.2 Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session__test.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.h (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage__test.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__test.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_gui_word__register__dialog_word__register__dialog.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_prediction_user__history__predictor.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_prediction_user__history__predictor.h (contents, props changed) head/japanese/mozc-server/files/patch-src_prediction_user__history__predictor__test.cc (contents, props changed) head/japanese/mozc-server/files/patch-src_rewriter_usage__rewriter__test.cc (contents, props changed) Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,11 @@ +--- src/dictionary/user_dictionary.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary.cc +@@ -289,7 +289,7 @@ class UserDictionary::UserDictionaryRelo + } + + auto_register_mode_ = false; +- dic_->Load(*(storage.get())); ++ dic_->Load(storage.get()->user_dictionary_storage_base); + } + + private: Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,200 @@ +--- src/dictionary/user_dictionary_session.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_session.cc +@@ -69,11 +69,11 @@ class UndoCreateDictionaryCommand : publ + } + + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { +- if (storage->dictionaries_size() == 0) { ++ if (storage->user_dictionary_storage_base.dictionaries_size() == 0) { + return false; + } + +- storage->mutable_dictionaries()->RemoveLast(); ++ storage->user_dictionary_storage_base.mutable_dictionaries()->RemoveLast(); + return true; + } + +@@ -94,7 +94,7 @@ class UndoDeleteDictionaryCommand : publ + } + + RepeatedPtrField *dictionaries = +- storage->mutable_dictionaries(); ++ storage->user_dictionary_storage_base.mutable_dictionaries(); + dictionaries->AddAllocated(dictionary_.release()); + + // Adjust the position of the reverted dictionary. +@@ -120,10 +120,10 @@ class UndoDeleteDictionaryWithEnsuringNo + } + + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { +- if (storage->dictionaries_size() != 1) { ++ if (storage->user_dictionary_storage_base.dictionaries_size() != 1) { + return false; + } +- dictionary_->Swap(storage->mutable_dictionaries(0)); ++ dictionary_->Swap(storage->user_dictionary_storage_base.mutable_dictionaries(0)); + return true; + } + +@@ -144,7 +144,7 @@ class UndoRenameDictionaryCommand : publ + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage, dictionary_id_); ++ &storage->user_dictionary_storage_base, dictionary_id_); + if (dictionary == NULL) { + return false; + } +@@ -169,7 +169,7 @@ class UndoAddEntryCommand : public UserD + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage, dictionary_id_); ++ &storage->user_dictionary_storage_base, dictionary_id_); + if (dictionary == NULL || dictionary->entries_size() == 0) { + return false; + } +@@ -195,7 +195,7 @@ class UndoEditEntryCommand : public User + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage, dictionary_id_); ++ &storage->user_dictionary_storage_base, dictionary_id_); + if (dictionary == NULL || + index_ < 0 || dictionary->entries_size() <= index_) { + return false; +@@ -240,7 +240,7 @@ class UndoDeleteEntryCommand : public Us + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage, dictionary_id_); ++ &storage->user_dictionary_storage_base, dictionary_id_); + if (dictionary == NULL) { + return false; + } +@@ -306,7 +306,7 @@ class UndoImportFromStringCommand : publ + virtual bool RunUndo(mozc::UserDictionaryStorage *storage) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage, dictionary_id_); ++ &storage->user_dictionary_storage_base, dictionary_id_); + if (dictionary == NULL) { + return false; + } +@@ -345,7 +345,7 @@ UserDictionarySession::~UserDictionarySe + + // TODO(hidehiko) move this to header. + const UserDictionaryStorage &UserDictionarySession::storage() const { +- return *storage_; ++ return storage_->user_dictionary_storage_base; + } + mozc::UserDictionaryStorage *UserDictionarySession::mutable_storage() { + return storage_.get(); +@@ -464,7 +464,7 @@ UserDictionaryCommandStatus::Status User + const string &dictionary_name, uint64 *new_dictionary_id) { + UserDictionaryCommandStatus::Status status = + UserDictionaryUtil::CreateDictionary( +- storage_.get(), dictionary_name, new_dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_name, new_dictionary_id); + if (status == UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS) { + AddUndoCommand(new UndoCreateDictionaryCommand); + } +@@ -488,7 +488,7 @@ UserDictionarySession::DeleteDictionaryI + int original_index; + UserDictionary *deleted_dictionary; + if (!UserDictionaryUtil::DeleteDictionary( +- storage_.get(), dictionary_id, ++ &storage_->user_dictionary_storage_base, dictionary_id, + &original_index, &deleted_dictionary)) { + // Failed to delete the dictionary. + return UserDictionaryCommandStatus::UNKNOWN_DICTIONARY_ID; +@@ -510,7 +510,7 @@ UserDictionaryCommandStatus::Status User + uint64 dictionary_id, const string &dictionary_name) { + string original_name; + const UserDictionary *dictionary = +- UserDictionaryUtil::GetUserDictionaryById(*storage_, dictionary_id); ++ UserDictionaryUtil::GetUserDictionaryById(storage_->user_dictionary_storage_base, dictionary_id); + if (dictionary != NULL) { + // Note that if dictionary is null, it means the dictionary_id is invalid + // so following RenameDictionary will fail, and error handling is done +@@ -547,7 +547,7 @@ UserDictionaryCommandStatus::Status User + uint64 dictionary_id, const UserDictionary::Entry &entry) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage_.get(), dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_id); + if (dictionary == NULL) { + return UserDictionaryCommandStatus::UNKNOWN_DICTIONARY_ID; + } +@@ -575,7 +575,7 @@ UserDictionaryCommandStatus::Status User + uint64 dictionary_id, int index, const UserDictionary::Entry &entry) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage_.get(), dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_id); + if (dictionary == NULL) { + return UserDictionaryCommandStatus::UNKNOWN_DICTIONARY_ID; + } +@@ -604,7 +604,7 @@ UserDictionaryCommandStatus::Status User + uint64 dictionary_id, const std::vector &index_list) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage_.get(), dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_id); + if (dictionary == NULL) { + return UserDictionaryCommandStatus::UNKNOWN_DICTIONARY_ID; + } +@@ -644,7 +644,7 @@ UserDictionaryCommandStatus::Status User + uint64 dictionary_id, const string &data) { + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage_.get(), dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_id); + if (dictionary == NULL) { + return UserDictionaryCommandStatus::UNKNOWN_DICTIONARY_ID; + } +@@ -699,7 +699,7 @@ UserDictionarySession::ImportToNewDictio + uint64 *new_dictionary_id) { + UserDictionaryCommandStatus::Status status = + UserDictionaryUtil::CreateDictionary( +- storage_.get(), dictionary_name, new_dictionary_id); ++ &storage_->user_dictionary_storage_base, dictionary_name, new_dictionary_id); + if (status != UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS) { + return status; + } +@@ -709,7 +709,7 @@ UserDictionarySession::ImportToNewDictio + + UserDictionary *dictionary = + UserDictionaryUtil::GetMutableUserDictionaryById( +- storage_.get(), *new_dictionary_id); ++ &storage_->user_dictionary_storage_base, *new_dictionary_id); + if (dictionary == NULL) { + // The dictionary should be always found. + return UserDictionaryCommandStatus::UNKNOWN_ERROR; +@@ -719,7 +719,7 @@ UserDictionarySession::ImportToNewDictio + } + + bool UserDictionarySession::EnsureNonEmptyStorage() { +- if (storage_->dictionaries_size() > 0) { ++ if (storage_->user_dictionary_storage_base.dictionaries_size() > 0) { + // The storage already has at least one dictionary. Do nothing. + return false; + } +@@ -728,7 +728,7 @@ bool UserDictionarySession::EnsureNonEmp + uint64 new_dictionary_id; + UserDictionaryCommandStatus::Status status = + UserDictionaryUtil::CreateDictionary( +- storage_.get(), default_dictionary_name_, &new_dictionary_id); ++ &storage_->user_dictionary_storage_base, default_dictionary_name_, &new_dictionary_id); + CHECK_EQ( + status, UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS); + return true; +@@ -755,7 +755,7 @@ void UserDictionarySession::AddUndoComma + + void UserDictionarySession::ClearDictionariesAndUndoHistory() { + ScopedUserDictionaryLocker l(storage_.get()); +- storage_->clear_dictionaries(); ++ storage_->user_dictionary_storage_base.clear_dictionaries(); + ClearUndoHistory(); + } + Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session__test.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__session__test.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,62 @@ +--- src/dictionary/user_dictionary_session_test.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_session_test.cc +@@ -95,12 +95,12 @@ TEST_F(UserDictionarySessionTest, SaveAn + + ASSERT_EQ(UserDictionaryCommandStatus::FILE_NOT_FOUND, session.Load()); + +- session.mutable_storage()->set_version(10); ++ session.mutable_storage()->user_dictionary_storage_base.set_version(10); + ASSERT_EQ(UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS, + session.Save()); + + // Clear once, in order to make sure that Load is actually working. +- session.mutable_storage()->Clear(); ++ session.mutable_storage()->user_dictionary_storage_base.Clear(); + ASSERT_EQ(UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS, + session.Load()); + +@@ -130,7 +130,7 @@ TEST_F(UserDictionarySessionTest, DISABL + + // Create huge dummy data. + { +- UserDictionaryStorage *storage = session.mutable_storage(); ++ UserDictionaryStorage *storage = &session.mutable_storage()->user_dictionary_storage_base; + for (int i = 0; i < 100; ++i) { + UserDictionary *dictionary = storage->add_dictionaries(); + for (int j = 0; j < 1000; ++j) { +@@ -147,7 +147,7 @@ TEST_F(UserDictionarySessionTest, DISABL + ASSERT_EQ(UserDictionaryCommandStatus::FILE_SIZE_LIMIT_EXCEEDED, + session.Save()); + +- session.mutable_storage()->Clear(); ++ session.mutable_storage()->user_dictionary_storage_base.Clear(); + ASSERT_EQ(UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS, + session.Load()); + +@@ -191,7 +191,7 @@ TEST_F(UserDictionarySessionTest, Create + + while (session.storage().dictionaries_size() < + ::mozc::UserDictionaryStorage::max_dictionary_size()) { +- session.mutable_storage()->add_dictionaries(); ++ session.mutable_storage()->user_dictionary_storage_base.add_dictionaries(); + } + EXPECT_EQ(UserDictionaryCommandStatus::DICTIONARY_SIZE_LIMIT_EXCEEDED, + session.CreateDictionary("dictionary 2", &dummy_dictionary_id)); +@@ -204,7 +204,7 @@ TEST_F(UserDictionarySessionTest, Delete + const uint64 kDummyId = 10; + { + UserDictionary *user_dictionary = +- session.mutable_storage()->add_dictionaries(); ++ session.mutable_storage()->user_dictionary_storage_base.add_dictionaries(); + user_dictionary->set_id(kDummyId); + } + +@@ -229,7 +229,7 @@ TEST_F(UserDictionarySessionTest, + const uint64 kDummyId = 10; + { + UserDictionary *user_dictionary = +- session.mutable_storage()->add_dictionaries(); ++ session.mutable_storage()->user_dictionary_storage_base.add_dictionaries(); + user_dictionary->set_id(kDummyId); + } + Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,193 @@ +--- src/dictionary/user_dictionary_storage.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_storage.cc +@@ -109,7 +109,7 @@ bool UserDictionaryStorage::LoadInternal + mozc::protobuf::io::IstreamInputStream zero_copy_input(&ifs); + mozc::protobuf::io::CodedInputStream decoder(&zero_copy_input); + decoder.SetTotalBytesLimit(kDefaultTotalBytesLimit, -1); +- if (!ParseFromCodedStream(&decoder)) { ++ if (!user_dictionary_storage_base.ParseFromCodedStream(&decoder)) { + LOG(ERROR) << "Failed to parse"; + if (!decoder.ConsumedEntireMessage() || !ifs.eof()) { + LOG(ERROR) << "ParseFromStream failed: file seems broken"; +@@ -137,11 +137,11 @@ bool UserDictionaryStorage::Load() { + } + + // Check dictionary id here. if id is 0, assign random ID. +- for (int i = 0; i < dictionaries_size(); ++i) { +- const UserDictionary &dict = dictionaries(i); ++ for (int i = 0; i < user_dictionary_storage_base.dictionaries_size(); ++i) { ++ const UserDictionary &dict = user_dictionary_storage_base.dictionaries(i); + if (dict.id() == 0) { +- mutable_dictionaries(i)->set_id( +- UserDictionaryUtil::CreateNewDictionaryId(*this)); ++ user_dictionary_storage_base.mutable_dictionaries(i)->set_id( ++ UserDictionaryUtil::CreateNewDictionaryId(this->user_dictionary_storage_base)); + } + } + +@@ -171,7 +171,7 @@ bool UserDictionaryStorage::Save() { + return false; + } + +- if (!SerializeToOstream(&ofs)) { ++ if (!user_dictionary_storage_base.SerializeToOstream(&ofs)) { + LOG(ERROR) << "SerializeToString failed"; + last_error_type_ = SYNC_FAILURE; + return false; +@@ -227,7 +227,7 @@ bool UserDictionaryStorage::ExportDictio + return false; + } + +- const UserDictionary &dic = dictionaries(index); ++ const UserDictionary &dic = user_dictionary_storage_base.dictionaries(index); + for (size_t i = 0; i < dic.entries_size(); ++i) { + const UserDictionaryEntry &entry = dic.entries(i); + ofs << entry.key() << "\t" << entry.value() << "\t" +@@ -241,7 +241,7 @@ bool UserDictionaryStorage::ExportDictio + bool UserDictionaryStorage::CreateDictionary( + const string &dic_name, uint64 *new_dic_id) { + UserDictionaryCommandStatus::Status status = +- UserDictionaryUtil::CreateDictionary(this, dic_name, new_dic_id); ++ UserDictionaryUtil::CreateDictionary(&this->user_dictionary_storage_base, dic_name, new_dic_id); + // Update last_error_type_ + switch (status) { + case UserDictionaryCommandStatus::DICTIONARY_NAME_EMPTY: +@@ -273,7 +273,7 @@ bool UserDictionaryStorage::CreateDictio + } + + bool UserDictionaryStorage::DeleteDictionary(uint64 dic_id) { +- if (!UserDictionaryUtil::DeleteDictionary(this, dic_id, NULL, NULL)) { ++ if (!UserDictionaryUtil::DeleteDictionary(&this->user_dictionary_storage_base, dic_id, NULL, NULL)) { + // Failed to delete dictionary. + last_error_type_ = INVALID_DICTIONARY_ID; + return false; +@@ -304,8 +304,8 @@ bool UserDictionaryStorage::RenameDictio + return true; + } + +- for (int i = 0; i < dictionaries_size(); ++i) { +- if (dic_name == dictionaries(i).name()) { ++ for (int i = 0; i < user_dictionary_storage_base.dictionaries_size(); ++i) { ++ if (dic_name == user_dictionary_storage_base.dictionaries(i).name()) { + last_error_type_ = DUPLICATED_DICTIONARY_NAME; + LOG(ERROR) << "duplicated dictionary name"; + return false; +@@ -318,14 +318,14 @@ bool UserDictionaryStorage::RenameDictio + } + + int UserDictionaryStorage::GetUserDictionaryIndex(uint64 dic_id) const { +- return UserDictionaryUtil::GetUserDictionaryIndexById(*this, dic_id); ++ return UserDictionaryUtil::GetUserDictionaryIndexById(this->user_dictionary_storage_base, dic_id); + } + + bool UserDictionaryStorage::GetUserDictionaryId(const string &dic_name, + uint64 *dic_id) { +- for (size_t i = 0; i < dictionaries_size(); ++i) { +- if (dic_name == dictionaries(i).name()) { +- *dic_id = dictionaries(i).id(); ++ for (size_t i = 0; i < user_dictionary_storage_base.dictionaries_size(); ++i) { ++ if (dic_name == user_dictionary_storage_base.dictionaries(i).name()) { ++ *dic_id = user_dictionary_storage_base.dictionaries(i).id(); + return true; + } + } +@@ -335,7 +335,7 @@ bool UserDictionaryStorage::GetUserDicti + + user_dictionary::UserDictionary *UserDictionaryStorage::GetUserDictionary( + uint64 dic_id) { +- return UserDictionaryUtil::GetMutableUserDictionaryById(this, dic_id); ++ return UserDictionaryUtil::GetMutableUserDictionaryById(&this->user_dictionary_storage_base, dic_id); + } + + UserDictionaryStorage::UserDictionaryStorageErrorType +@@ -352,8 +352,8 @@ bool UserDictionaryStorage::AddToAutoReg + } + + int auto_index = -1; +- for (int i = 0; i < dictionaries_size(); ++i) { +- if (dictionaries(i).name() == kAutoRegisteredDictionaryName) { ++ for (int i = 0; i < user_dictionary_storage_base.dictionaries_size(); ++i) { ++ if (user_dictionary_storage_base.dictionaries(i).name() == kAutoRegisteredDictionaryName) { + auto_index = i; + break; + } +@@ -361,17 +361,17 @@ bool UserDictionaryStorage::AddToAutoReg + + UserDictionary *dic = NULL; + if (auto_index == -1) { +- if (UserDictionaryUtil::IsStorageFull(*this)) { ++ if (UserDictionaryUtil::IsStorageFull(this->user_dictionary_storage_base)) { + last_error_type_ = TOO_MANY_DICTIONARIES; + LOG(ERROR) << "too many dictionaries"; + UnLock(); + return false; + } +- dic = add_dictionaries(); +- dic->set_id(UserDictionaryUtil::CreateNewDictionaryId(*this)); ++ dic = user_dictionary_storage_base.add_dictionaries(); ++ dic->set_id(UserDictionaryUtil::CreateNewDictionaryId(this->user_dictionary_storage_base)); + dic->set_name(kAutoRegisteredDictionaryName); + } else { +- dic = mutable_dictionaries(auto_index); ++ dic = user_dictionary_storage_base.mutable_dictionaries(auto_index); + } + + if (dic == NULL) { +@@ -410,13 +410,13 @@ bool UserDictionaryStorage::AddToAutoReg + } + + bool UserDictionaryStorage::ConvertSyncDictionariesToNormalDictionaries() { +- if (CountSyncableDictionaries(*this) == 0) { ++ if (CountSyncableDictionaries(this->user_dictionary_storage_base) == 0) { + return false; + } + +- for (int dictionary_index = dictionaries_size() - 1; ++ for (int dictionary_index = user_dictionary_storage_base.dictionaries_size() - 1; + dictionary_index >= 0; --dictionary_index) { +- UserDictionary *dic = mutable_dictionaries(dictionary_index); ++ UserDictionary *dic = user_dictionary_storage_base.mutable_dictionaries(dictionary_index); + if (!dic->syncable()) { + continue; + } +@@ -433,10 +433,10 @@ bool UserDictionaryStorage::ConvertSyncD + + // Delete removed or unused sync dictionaries. + if (dic->removed() || dic->entries_size() == 0) { +- for (int i = dictionary_index + 1; i < dictionaries_size(); ++i) { +- mutable_dictionaries()->SwapElements(i - 1, i); ++ for (int i = dictionary_index + 1; i < user_dictionary_storage_base.dictionaries_size(); ++i) { ++ user_dictionary_storage_base.mutable_dictionaries()->SwapElements(i - 1, i); + } +- mutable_dictionaries()->RemoveLast(); ++ user_dictionary_storage_base.mutable_dictionaries()->RemoveLast(); + continue; + } + +@@ -445,7 +445,7 @@ bool UserDictionaryStorage::ConvertSyncD + kDictionaryNameConvertedFromSyncableDictionary; + int index = 0; + while (UserDictionaryUtil::ValidateDictionaryName( +- *this, new_dictionary_name) ++ this->user_dictionary_storage_base, new_dictionary_name) + != UserDictionaryCommandStatus::USER_DICTIONARY_COMMAND_SUCCESS) { + ++index; + new_dictionary_name = Util::StringPrintf( +@@ -456,7 +456,7 @@ bool UserDictionaryStorage::ConvertSyncD + dic->set_syncable(false); + } + +- DCHECK_EQ(0, CountSyncableDictionaries(*this)); ++ DCHECK_EQ(0, CountSyncableDictionaries(this->user_dictionary_storage_base)); + + return true; + } +@@ -487,7 +487,7 @@ size_t UserDictionaryStorage::max_dictio + bool UserDictionaryStorage::IsValidDictionaryName(const string &name) { + UserDictionaryCommandStatus::Status status = + UserDictionaryUtil::ValidateDictionaryName( +- UserDictionaryStorage::default_instance(), name); ++ user_dictionary::UserDictionaryStorage::default_instance(), name); + + // Update last_error_type_. + switch (status) { Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.h ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage.h Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,21 @@ +--- src/dictionary/user_dictionary_storage.h.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_storage.h +@@ -72,13 +72,15 @@ namespace mozc { + class Mutex; + class ProcessMutex; + +-// Inherit from ProtocolBuffer +-// TODO(hidehiko): Get rid of this implementation. +-class UserDictionaryStorage : public user_dictionary::UserDictionaryStorage { ++class UserDictionaryStorage { + public: + typedef user_dictionary::UserDictionary UserDictionary; + typedef user_dictionary::UserDictionary::Entry UserDictionaryEntry; + ++ // Instance of base class generated by Protocol Buffers compiler. ++ // Regular inheritance strongly discouraged. ++ user_dictionary::UserDictionaryStorage user_dictionary_storage_base; ++ + enum UserDictionaryStorageErrorType { + USER_DICTIONARY_STORAGE_NO_ERROR = 0, // default + FILE_NOT_EXISTS, Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage__test.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__storage__test.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,188 @@ +--- src/dictionary/user_dictionary_storage_test.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_storage_test.cc +@@ -118,13 +118,13 @@ TEST_F(UserDictionaryStorageTest, BasicO + const size_t kDictionariesSize = 3; + uint64 id[kDictionariesSize]; + +- const size_t dict_size = storage.dictionaries_size(); ++ const size_t dict_size = storage.user_dictionary_storage_base.dictionaries_size(); + + for (size_t i = 0; i < kDictionariesSize; ++i) { + EXPECT_TRUE(storage.CreateDictionary( + "test" + std::to_string(static_cast(i)), + &id[i])); +- EXPECT_EQ(i + 1 + dict_size, storage.dictionaries_size()); ++ EXPECT_EQ(i + 1 + dict_size, storage.user_dictionary_storage_base.dictionaries_size()); + } + + for (size_t i = 0; i < kDictionariesSize; ++i) { +@@ -133,7 +133,7 @@ TEST_F(UserDictionaryStorageTest, BasicO + } + + for (size_t i = 0; i < kDictionariesSize; ++i) { +- EXPECT_EQ(storage.mutable_dictionaries(i + dict_size), ++ EXPECT_EQ(storage.user_dictionary_storage_base.mutable_dictionaries(i + dict_size), + storage.GetUserDictionary(id[i])); + EXPECT_EQ(NULL, storage.GetUserDictionary(id[i] + 1)); + } +@@ -165,7 +165,7 @@ TEST_F(UserDictionaryStorageTest, BasicO + EXPECT_FALSE(storage.DeleteDictionary(0)); + + EXPECT_TRUE(storage.DeleteDictionary(id[1])); +- EXPECT_EQ(kDictionariesSize + dict_size - 1, storage.dictionaries_size()); ++ EXPECT_EQ(kDictionariesSize + dict_size - 1, storage.user_dictionary_storage_base.dictionaries_size()); + } + + TEST_F(UserDictionaryStorageTest, DeleteTest) { +@@ -174,7 +174,7 @@ TEST_F(UserDictionaryStorageTest, Delete + + // repeat 10 times + for (int i = 0; i < 10; ++i) { +- storage.Clear(); ++ storage.user_dictionary_storage_base.Clear(); + std::vector ids(100); + for (size_t i = 0; i < ids.size(); ++i) { + EXPECT_TRUE(storage.CreateDictionary( +@@ -191,10 +191,10 @@ TEST_F(UserDictionaryStorageTest, Delete + alive.push_back(ids[i]); + } + +- EXPECT_EQ(alive.size(), storage.dictionaries_size()); ++ EXPECT_EQ(alive.size(), storage.user_dictionary_storage_base.dictionaries_size()); + + for (size_t i = 0; i < alive.size(); ++i) { +- EXPECT_EQ(alive[i], storage.dictionaries(i).id()); ++ EXPECT_EQ(alive[i], storage.user_dictionary_storage_base.dictionaries(i).id()); + } + } + } +@@ -268,7 +268,7 @@ TEST_F(UserDictionaryStorageTest, Serial + const size_t entry_size = Util::Random(100) + 1; + for (size_t j = 0; j < entry_size; ++j) { + UserDictionaryStorage::UserDictionary *dic = +- storage1.mutable_dictionaries(i); ++ storage1.user_dictionary_storage_base.mutable_dictionaries(i); + UserDictionaryStorage::UserDictionaryEntry *entry = + dic->add_entries(); + entry->set_key(GenRandomString(10)); +@@ -288,7 +288,7 @@ TEST_F(UserDictionaryStorageTest, Serial + EXPECT_TRUE(storage2.Load()); + } + +- EXPECT_EQ(storage1.DebugString(), storage2.DebugString()); ++ EXPECT_EQ(storage1.user_dictionary_storage_base.DebugString(), storage2.user_dictionary_storage_base.DebugString()); + } + } + +@@ -352,7 +352,7 @@ TEST_F(UserDictionaryStorageTest, Conver + uint64 dict_id = 0; + ASSERT_TRUE(storage.CreateDictionary(data.dictionary_name, &dict_id)); + UserDictionaryStorage::UserDictionary *dict = +- storage.mutable_dictionaries(storage.GetUserDictionaryIndex(dict_id)); ++ storage.user_dictionary_storage_base.mutable_dictionaries(storage.GetUserDictionaryIndex(dict_id)); + dict->set_syncable(data.is_sync_dictionary); + dict->set_removed(data.is_removed_dictionary); + if (data.has_normal_entry) { +@@ -369,7 +369,7 @@ TEST_F(UserDictionaryStorageTest, Conver + entry->set_removed(true); + } + } +- EXPECT_EQ(9, UserDictionaryStorage::CountSyncableDictionaries(storage)); ++ EXPECT_EQ(9, UserDictionaryStorage::CountSyncableDictionaries(storage.user_dictionary_storage_base)); + + ASSERT_TRUE(storage.ConvertSyncDictionariesToNormalDictionaries()); + +@@ -385,12 +385,12 @@ TEST_F(UserDictionaryStorageTest, Conver + { true, kDictionaryNameConvertedFromSyncableDictionary }, + }; + +- EXPECT_EQ(0, UserDictionaryStorage::CountSyncableDictionaries(storage)); +- ASSERT_EQ(arraysize(expected_data), storage.dictionaries_size()); ++ EXPECT_EQ(0, UserDictionaryStorage::CountSyncableDictionaries(storage.user_dictionary_storage_base)); ++ ASSERT_EQ(arraysize(expected_data), storage.user_dictionary_storage_base.dictionaries_size()); + for (size_t i = 0; i < arraysize(expected_data); ++i) { + SCOPED_TRACE(Util::StringPrintf("verify %d", static_cast(i))); + const ExpectedData &expected = expected_data[i]; +- const UserDictionaryStorage::UserDictionary &dict = storage.dictionaries(i); ++ const UserDictionaryStorage::UserDictionary &dict = storage.user_dictionary_storage_base.dictionaries(i); + + EXPECT_EQ(expected.dictionary_name, dict.name()); + EXPECT_FALSE(dict.syncable()); +@@ -404,42 +404,42 @@ TEST_F(UserDictionaryStorageTest, Conver + } + + // Test duplicated dictionary name. +- storage.Clear(); ++ storage.user_dictionary_storage_base.Clear(); + { + uint64 dict_id = 0; + storage.CreateDictionary( + UserDictionaryStorage::default_sync_dictionary_name(), &dict_id); + storage.CreateDictionary( + kDictionaryNameConvertedFromSyncableDictionary, &dict_id); +- ASSERT_EQ(2, storage.dictionaries_size()); ++ ASSERT_EQ(2, storage.user_dictionary_storage_base.dictionaries_size()); + UserDictionaryStorage::UserDictionary *dict; +- dict = storage.mutable_dictionaries(0); ++ dict = storage.user_dictionary_storage_base.mutable_dictionaries(0); + dict->set_syncable(true); + dict->add_entries()->set_key("0"); +- dict = storage.mutable_dictionaries(1); ++ dict = storage.user_dictionary_storage_base.mutable_dictionaries(1); + dict->set_syncable(false); + dict->add_entries()->set_key("1"); + } + ASSERT_TRUE(storage.ConvertSyncDictionariesToNormalDictionaries()); +- EXPECT_EQ(0, UserDictionaryStorage::CountSyncableDictionaries(storage)); +- EXPECT_EQ(2, storage.dictionaries_size()); ++ EXPECT_EQ(0, UserDictionaryStorage::CountSyncableDictionaries(storage.user_dictionary_storage_base)); ++ EXPECT_EQ(2, storage.user_dictionary_storage_base.dictionaries_size()); + EXPECT_EQ(Util::StringPrintf("%s_1", + kDictionaryNameConvertedFromSyncableDictionary), +- storage.dictionaries(0).name()); ++ storage.user_dictionary_storage_base.dictionaries(0).name()); + EXPECT_EQ(kDictionaryNameConvertedFromSyncableDictionary, +- storage.dictionaries(1).name()); ++ storage.user_dictionary_storage_base.dictionaries(1).name()); + } + + TEST_F(UserDictionaryStorageTest, AddToAutoRegisteredDictionary) { + { + UserDictionaryStorage storage(GetUserDictionaryFile()); +- EXPECT_EQ(0, storage.dictionaries_size()); ++ EXPECT_EQ(0, storage.user_dictionary_storage_base.dictionaries_size()); + EXPECT_TRUE(storage.AddToAutoRegisteredDictionary( + "key1", "value1", UserDictionary::NOUN)); +- EXPECT_EQ(1, storage.dictionaries_size()); +- EXPECT_EQ(1, storage.dictionaries(0).entries_size()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries_size()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries(0).entries_size()); + const UserDictionaryStorage::UserDictionaryEntry &entry1 = +- storage.dictionaries(0).entries(0); ++ storage.user_dictionary_storage_base.dictionaries(0).entries(0); + EXPECT_EQ("key1", entry1.key()); + EXPECT_EQ("value1", entry1.value()); + EXPECT_EQ(UserDictionary::NOUN, entry1.pos()); +@@ -447,10 +447,10 @@ TEST_F(UserDictionaryStorageTest, AddToA + + EXPECT_TRUE(storage.AddToAutoRegisteredDictionary( + "key2", "value2", UserDictionary::NOUN)); +- EXPECT_EQ(1, storage.dictionaries_size()); +- EXPECT_EQ(2, storage.dictionaries(0).entries_size()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries_size()); ++ EXPECT_EQ(2, storage.user_dictionary_storage_base.dictionaries(0).entries_size()); + const UserDictionaryStorage::UserDictionaryEntry &entry2 = +- storage.dictionaries(0).entries(1); ++ storage.user_dictionary_storage_base.dictionaries(0).entries(1); + EXPECT_EQ("key2", entry2.key()); + EXPECT_EQ("value2", entry2.value()); + EXPECT_EQ(UserDictionary::NOUN, entry2.pos()); +@@ -474,7 +474,7 @@ TEST_F(UserDictionaryStorageTest, Export + { + UserDictionaryStorage storage(GetUserDictionaryFile()); + { +- UserDictionary *dictionary = storage.add_dictionaries(); ++ UserDictionary *dictionary = storage.user_dictionary_storage_base.add_dictionaries(); + dictionary->set_id(kDummyDictionaryId); + UserDictionary::Entry *entry = dictionary->add_entries(); + entry->set_key("key"); Added: head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__test.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_dictionary_user__dictionary__test.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,229 @@ +--- src/dictionary/user_dictionary_test.cc.orig 2019-03-04 18:35:55 UTC ++++ src/dictionary/user_dictionary_test.cc +@@ -334,9 +334,9 @@ class UserDictionaryTest : public ::test + std::istringstream is(contents); + CHECK(is.good()); + +- storage->Clear(); ++ storage->user_dictionary_storage_base.Clear(); + UserDictionaryStorage::UserDictionary *dic +- = storage->add_dictionaries(); ++ = storage->user_dictionary_storage_base.add_dictionaries(); + CHECK(dic); + + string line; +@@ -389,7 +389,7 @@ TEST_F(UserDictionaryTest, TestLookupPre + { + UserDictionaryStorage storage(""); + UserDictionaryTest::LoadFromString(kUserDictionary0, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // A normal lookup operation. +@@ -425,7 +425,7 @@ TEST_F(UserDictionaryTest, TestLookupPre + { + UserDictionaryStorage storage(""); + LoadFromString(kUserDictionary1, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // A normal lookup again. +@@ -450,7 +450,7 @@ TEST_F(UserDictionaryTest, TestLookupPre + { + UserDictionaryStorage storage(""); + LoadFromString(kUserDictionary0, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // A normal lookup operation. +@@ -480,7 +480,7 @@ TEST_F(UserDictionaryTest, TestLookupPre + { + UserDictionaryStorage storage(""); + LoadFromString(kUserDictionary1, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // A normal lookup. +@@ -504,7 +504,7 @@ TEST_F(UserDictionaryTest, TestLookupExa + { + UserDictionaryStorage storage(""); + LoadFromString(kUserDictionary0, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // A normal lookup operation. +@@ -540,7 +540,7 @@ TEST_F(UserDictionaryTest, TestLookupExa + uint64 id = 0; + EXPECT_TRUE(storage.CreateDictionary("test", &id)); + UserDictionaryStorage::UserDictionary *dic = +- storage.mutable_dictionaries(0); ++ storage.user_dictionary_storage_base.mutable_dictionaries(0); + + // "名詞" + UserDictionaryStorage::UserDictionaryEntry *entry = +@@ -555,7 +555,7 @@ TEST_F(UserDictionaryTest, TestLookupExa + entry->set_value("suggest_only"); + entry->set_pos(user_dictionary::UserDictionary::SUGGESTION_ONLY); + +- user_dic->Load(storage); ++ user_dic->Load(storage.user_dictionary_storage_base); + } + + // "suggestion_only" should not be looked up. +@@ -577,7 +577,7 @@ TEST_F(UserDictionaryTest, IncognitoMode + { + UserDictionaryStorage storage(""); + UserDictionaryTest::LoadFromString(kUserDictionary0, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + TestLookupPrefixHelper(nullptr, 0, "start", 4, *dic); +@@ -612,7 +612,7 @@ TEST_F(UserDictionaryTest, AsyncLoadTest + uint64 id = 0; + EXPECT_TRUE(storage.CreateDictionary("test", &id)); + UserDictionaryStorage::UserDictionary *dic = +- storage.mutable_dictionaries(0); ++ storage.user_dictionary_storage_base.mutable_dictionaries(0); + for (size_t j = 0; j < 10000; ++j) { + UserDictionaryStorage::UserDictionaryEntry *entry = + dic->add_entries(); +@@ -681,15 +681,15 @@ TEST_F(UserDictionaryTest, AddToAutoRegi + UserDictionaryStorage storage(filename); + EXPECT_TRUE(storage.Load()); + int index = 0; +- EXPECT_EQ(1, storage.dictionaries_size()); +- EXPECT_EQ(100, storage.dictionaries(index).entries_size()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries_size()); ++ EXPECT_EQ(100, storage.user_dictionary_storage_base.dictionaries(index).entries_size()); + for (int i = 0; i < 100; ++i) { + EXPECT_EQ("key" + std::to_string(i), +- storage.dictionaries(index).entries(i).key()); ++ storage.user_dictionary_storage_base.dictionaries(index).entries(i).key()); + EXPECT_EQ("value" + std::to_string(i), +- storage.dictionaries(index).entries(i).value()); ++ storage.user_dictionary_storage_base.dictionaries(index).entries(i).value()); + EXPECT_EQ(user_dictionary::UserDictionary::NOUN, +- storage.dictionaries(index).entries(i).pos()); ++ storage.user_dictionary_storage_base.dictionaries(index).entries(i).pos()); + } + } + +@@ -722,12 +722,12 @@ TEST_F(UserDictionaryTest, AddToAutoRegi + { + UserDictionaryStorage storage(filename); + EXPECT_TRUE(storage.Load()); +- EXPECT_EQ(1, storage.dictionaries_size()); +- EXPECT_EQ(1, storage.dictionaries(0).entries_size()); +- EXPECT_EQ("key", storage.dictionaries(0).entries(0).key()); +- EXPECT_EQ("value", storage.dictionaries(0).entries(0).value()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries_size()); ++ EXPECT_EQ(1, storage.user_dictionary_storage_base.dictionaries(0).entries_size()); ++ EXPECT_EQ("key", storage.user_dictionary_storage_base.dictionaries(0).entries(0).key()); ++ EXPECT_EQ("value", storage.user_dictionary_storage_base.dictionaries(0).entries(0).value()); + EXPECT_EQ(user_dictionary::UserDictionary::NOUN, +- storage.dictionaries(0).entries(0).pos()); ++ storage.user_dictionary_storage_base.dictionaries(0).entries(0).pos()); + } + } + +@@ -746,7 +746,7 @@ TEST_F(UserDictionaryTest, TestSuppressi + uint64 id = 0; + EXPECT_TRUE(storage.CreateDictionary("test", &id)); + UserDictionaryStorage::UserDictionary *dic = +- storage.mutable_dictionaries(0); ++ storage.user_dictionary_storage_base.mutable_dictionaries(0); + for (size_t j = 0; j < 10000; ++j) { + UserDictionaryStorage::UserDictionaryEntry *entry = + dic->add_entries(); +@@ -770,7 +770,7 @@ TEST_F(UserDictionaryTest, TestSuppressi + + suppression_dictionary_->Lock(); + EXPECT_TRUE(suppression_dictionary_->IsLocked()); +- user_dic->Load(storage); ++ user_dic->Load(storage.user_dictionary_storage_base); + EXPECT_FALSE(suppression_dictionary_->IsLocked()); + + for (size_t j = 0; j < 10; ++j) { +@@ -782,11 +782,11 @@ TEST_F(UserDictionaryTest, TestSuppressi + + // Remove suppression entry + { +- storage.Clear(); ++ storage.user_dictionary_storage_base.Clear(); + uint64 id = 0; + EXPECT_TRUE(storage.CreateDictionary("test", &id)); + UserDictionaryStorage::UserDictionary *dic = +- storage.mutable_dictionaries(0); ++ storage.user_dictionary_storage_base.mutable_dictionaries(0); + for (size_t j = 0; j < 10000; ++j) { + UserDictionaryStorage::UserDictionaryEntry *entry = + dic->add_entries(); +@@ -798,7 +798,7 @@ TEST_F(UserDictionaryTest, TestSuppressi + } + + suppression_dictionary_->Lock(); +- user_dic->Load(storage); ++ user_dic->Load(storage.user_dictionary_storage_base); + EXPECT_FALSE(suppression_dictionary_->IsLocked()); + + for (size_t j = 0; j < 10; ++j) { +@@ -825,7 +825,7 @@ TEST_F(UserDictionaryTest, TestSuggestio + uint64 id = 0; + EXPECT_TRUE(storage.CreateDictionary("test", &id)); + UserDictionaryStorage::UserDictionary *dic = +- storage.mutable_dictionaries(0); ++ storage.user_dictionary_storage_base.mutable_dictionaries(0); + + for (size_t j = 0; j < 10; ++j) { + UserDictionaryStorage::UserDictionaryEntry *entry = +@@ -845,7 +845,7 @@ TEST_F(UserDictionaryTest, TestSuggestio + entry->set_pos(user_dictionary::UserDictionary::SUGGESTION_ONLY); + } + +- user_dic->Load(storage); ++ user_dic->Load(storage.user_dictionary_storage_base); + } + + { +@@ -878,7 +878,7 @@ TEST_F(UserDictionaryTest, TestUsageStat + UserDictionaryStorage storage(""); + + { +- UserDictionaryStorage::UserDictionary *dic1 = storage.add_dictionaries(); ++ UserDictionaryStorage::UserDictionary *dic1 = storage.user_dictionary_storage_base.add_dictionaries(); + CHECK(dic1); + UserDictionaryStorage::UserDictionaryEntry *entry; + entry = dic1->add_entries(); +@@ -893,7 +893,7 @@ TEST_F(UserDictionaryTest, TestUsageStat + entry->set_pos(user_dictionary::UserDictionary::NOUN); + } + { +- UserDictionaryStorage::UserDictionary *dic2 = storage.add_dictionaries(); ++ UserDictionaryStorage::UserDictionary *dic2 = storage.user_dictionary_storage_base.add_dictionaries(); + CHECK(dic2); + UserDictionaryStorage::UserDictionaryEntry *entry; + entry = dic2->add_entries(); +@@ -912,7 +912,7 @@ TEST_F(UserDictionaryTest, TestUsageStat + entry->set_value("value5"); + entry->set_pos(user_dictionary::UserDictionary::NOUN); + } +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + + EXPECT_INTEGER_STATS("UserRegisteredWord", 5); + } +@@ -925,7 +925,7 @@ TEST_F(UserDictionaryTest, LookupComment + { + UserDictionaryStorage storage(""); + UserDictionaryTest::LoadFromString(kUserDictionary0, &storage); +- dic->Load(storage); ++ dic->Load(storage.user_dictionary_storage_base); + } + + // Entry is in user dictionary but has no comment. Added: head/japanese/mozc-server/files/patch-src_gui_word__register__dialog_word__register__dialog.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_gui_word__register__dialog_word__register__dialog.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,11 @@ +--- src/gui/word_register_dialog/word_register_dialog.cc.orig 2019-03-04 18:35:55 UTC ++++ src/gui/word_register_dialog/word_register_dialog.cc +@@ -298,7 +298,7 @@ WordRegisterDialog::ErrorCode WordRegist + } + + UserDictionary *dic = +- session_->mutable_storage()->mutable_dictionaries(index); ++ session_->mutable_storage()->user_dictionary_storage_base.mutable_dictionaries(index); + CHECK(dic); + + if (dic->name() != DictionarycomboBox->currentText().toStdString()) { Added: head/japanese/mozc-server/files/patch-src_prediction_user__history__predictor.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/japanese/mozc-server/files/patch-src_prediction_user__history__predictor.cc Wed Jun 3 18:05:29 2020 (r537632) @@ -0,0 +1,63 @@ +--- src/prediction/user_history_predictor.cc.orig 2019-03-04 18:35:55 UTC ++++ src/prediction/user_history_predictor.cc +@@ -291,23 +291,23 @@ bool UserHistoryStorage::Load() { + return false; + } + +- if (!ParseFromString(input)) { ++ if (!user_history_base.ParseFromString(input)) { + LOG(ERROR) << "ParseFromString failed. message looks broken"; + return false; + } + +- VLOG(1) << "Loaded user histroy, size=" << entries_size(); ++ VLOG(1) << "Loaded user histroy, size=" << user_history_base.entries_size(); + return true; + } + + bool UserHistoryStorage::Save() const { +- if (entries_size() == 0) { ++ if (user_history_base.entries_size() == 0) { + LOG(WARNING) << "etries size is 0. Not saved"; + return false; + } + + string output; +- if (!AppendToString(&output)) { ++ if (!user_history_base.AppendToString(&output)) { + LOG(ERROR) << "AppendToString failed"; + return false; + } +@@ -494,12 +494,12 @@ bool UserHistoryPredictor::Load() { + return false; + } + +- for (size_t i = 0; i < history.entries_size(); ++i) { +- dic_->Insert(EntryFingerprint(history.entries(i)), +- history.entries(i)); ++ for (size_t i = 0; i < history.user_history_base.entries_size(); ++i) { ++ dic_->Insert(EntryFingerprint(history.user_history_base.entries(i)), ++ history.user_history_base.entries(i)); + } + +- VLOG(1) << "Loaded user histroy, size=" << history.entries_size(); ++ VLOG(1) << "Loaded user histroy, size=" << history.user_history_base.entries_size(); + + return true; + } +@@ -521,13 +521,13 @@ bool UserHistoryPredictor::Save() { + + UserHistoryStorage history(filename); + for (const DicElement *elm = tail; elm != nullptr; elm = elm->prev) { +- history.add_entries()->CopyFrom(elm->value); ++ history.user_history_base.add_entries()->CopyFrom(elm->value); + } + + // Updates usage stats here. + UsageStats::SetInteger( + "UserHistoryPredictorEntrySize", *** DIFF OUTPUT TRUNCATED AT 1000 LINES ***