无法基于最新的 opencc 1.1.0 编译

[huyz-git]

opencc最近发布了1.1.0版：https://github.com/BYVoid/OpenCC/releases/tag/ver.1.1.0
但是，如果用opencc 1.1.0版来编译librime，则会报错：

/tmp/librime/trunk/src/librime-1.5.3/src/rime/gear/simplifier.cc: 在成员函数‘bool rime::Opencc::ConvertWord(const string&, std::vector<std::__cxx11::basic_string<char> >*)’中:
/tmp/librime/trunk/src/librime-1.5.3/src/rime/gear/simplifier.cc:57:46: 错误：cannot convert ‘std::__cxx11::basic_string<char>’ to ‘const char*’ in initialization
   57 |       for (const char* value : entry->Values()) {
      |                                              ^
make[2]: *** [src/CMakeFiles/rime.dir/build.make:1214：src/CMakeFiles/rime.dir/rime/gear/simplifier.cc.o] 错误 1
make[2]: *** 正在等待未完成的任务....
make[1]: *** [CMakeFiles/Makefile2:213：src/CMakeFiles/rime.dir/all] 错误 2
make: *** [Makefile:161：all] 错误 2

编译指令为

export CXXFLAG=" -DNDEBUG"
cmake . -Bbuild -DCMAKE_INSTALL_PREFIX=/usr
cmake --build build

使用 opencc 1.0.6 版则可以正常编译。

操作系统：Archlinux
其余依赖的版本：
gcc 10.1
boost 1.72.0
glog 0.4.0
leveldb 1.22
marisa 0.2.5
yaml-cpp 0.6.3
cmake 3.17.2
gtest 1.8.1

[huyz-git]

把simplifier.cc的第57行

for (const char* value : entry->Values()) {

改成

for (auto value : entry->Values()) {

能编译通过。

有没有什么副作用还不清楚。

[lotem]

simplifier.cc 的這處代碼 entry->Values() 用到 opencc::DictEntry::Values().
opencc 1.1.0 的定義是 virtual vector<std::string> Values() const = 0;
以前版本的定義是返回 vector<const char*>

因爲對編譯器不夠熟悉，我現在無法判斷以下哪種寫法既能兼容舊版本的opencc接口，又能在不同版本中儘量避免性能損失。

// 舊版本opencc循環變量`value`是 `const char*` 指針，新版本則直觀上是從返回的容器中複製了`string`到`value`變量，不知編譯器能否繼續優化。
for (auto value : entry->Values()) { my_string_vector->push_back(value); }

// 這個寫法等於是舊版opencc顯式做了由C字符串構造`string`。新版opencc等同於上面一種寫法，不知能否自動優化掉兩個容器間多餘的一次複製。
for (string value : entry->Values()) { my_string_vector->push_back(value); }

// 這個是比較保險的C++03寫法，確定會發生一次複製，然而 `const&` 引用會不會阻止更厲害的優化呢……
for (const auto& value : entry->Values()) { my_string_vector->push_back(value); }

// 最後這種寫法可否對新版opecc通過右值引用直接把字符串移動到 `my_string_vector` 容器中？
for (auto&& value : entry->Values()) { my_string_vector->push_back(std::move(value)); }

@BYVoid

[BYVoid]

auto&& 應該沒問題。1.1.0因爲引入Marisa，數據格式發生改變，難以繼續直接引用內存地址。不過之後應該還可以改進，代價是初始化時間。

[wengxt]

auto&& 應該沒問題。1.1.0因爲引入Marisa，數據格式發生改變，難以繼續直接引用內存地址。不過之後應該還可以改進，代價是初始化時間。

改归改别忘了改 soversion 啊 ´_>` ……

[BYVoid]

Ok，改了之後我發佈一下1.1.0-1吧。

[BYVoid]

爲了保持版本號一致性，我準備將 SOVERSION 更新到 1.1，現在是2。

[BYVoid]

BYVoid/OpenCC@c6cde22

[BYVoid]

opencc 1.1.1 已經發佈。

[wengxt]

指出一个错误理解 SO_VERSION 和你的主版本号没有必要一致。
尽管 https://semver.org/ 你的 soversion 可以和 major 一致，但这仅限于你完全按照要求没有 break abi 。

每当你 abi break ，你的 so version 应当加一，而且永远不应该倒退。

我没有看你具体的修改，但是我感觉你可能实际犯了一个错误…

[BYVoid]

SOVERSION從2到1.1不是倒退，而是從一位增加到了2位。