To CZChen:
這次我用 cherry-pick 整理了,所有 init_database 修改都在上面
有些衝突是 pick 時發生的,我目前先保留它的訊息
https://github.com/school510587/general_index_libchewing/tree/rdb_review
關於建議回應如下:
commit message 我已經盡量以我的理解去遵守。
Style 部分,我全盲只能用點顯器閱讀,視野只有 40 characters (13~20 Chinese words),
所以多空格反而會讓我不便閱讀,我會不自覺不想多打 style 用的空白,請見諒!
我已經將 if, while, for, switch 與 ( 間嚴格空一格,{ 與上述敘述需同行
if, while, for 子敘述 tab 進去、等號即比較算符兩邊空白,這兩問題我盡量抓漏了
cmake 部分,這新 branch 我可以在 libchewing/ 成功測試,可是 test/ 下失敗
昨天那支看錯誤懷疑少了某個 #define 讓 plat_mmap_posix.h, plat_mmap_windows.h 都沒有 include
今天這支無法建立 am--refresh, 我不清楚問題原因,抱歉還請指點
最後辭發音檢查,我用 PCMan 無法操作含有 UTF-8 的段落,故無法恢復 exception_phrase,
且之前討論你說讓字典決定即可,故我在此忽略所有發音錯誤,即 exception_phrase.
Pull request 部分,我看了
http://ihower.tw/blog/archives/7560
但 brew install hub / gem install hub 似乎都沒用,我可以如何做到這要求?
Thanks!
在 2013/8/26,ChangZhuo Chen <
czc...@gmail.com> 撰寫:
> Hi,
>
> 幾點建議如下:
>
>
> - code 的部份可以使用 github 的 pull request 嗎?這樣 review 比較方便。
> - cmake build 的 unit test 有問題,travis-ci 的結果如 [1],這部份要再檢查看看。
> - git log 部份有辦法 rebase 讓你的修改在最上面嗎?目前 history 有點亂,而且還跨過 v0.3.5 的
> release,這會造成之後 maintain 上的困難。
> - git commit message 請遵守 [2]。
> - init_database.c 的 coding style 有些不一致,像是 if, +=, =, () 左右的空格,{
> 的位置等。這部份建議用 K&R style [3]。
> - init_database.c 內目前沒有對 "詞" 作發音檢查?這部份會造成詞庫正確性出問題,請務必把 "詞" 的發音檢查加回去。
> --
> 您已訂閱「Google 網上論壇」的「Chewing IM Development」群組,因此我們特別傳送這封郵件通知您。
> 如要取消訂閱這個群組並停止接收來自這個群組的郵件,請傳送電子郵件到
>
chewing-deve...@googlegroups.com。
> 如要在此群組張貼留言,請傳送電子郵件至
chewin...@googlegroups.com。
> 請前往以下網址造訪這個群組:
http://groups.google.com/group/chewing-devel。
> 如需更多選項,請前往:
https://groups.google.com/groups/opt_out。
>