Kouji Matsui
Kouji Matsui
@komutan ドキュメントですが、 `MeCab.DotNet` 側にはドキュメントと呼べるようなものがない 😄 ので、[今の `NMeCab` の最初のサンプルコード](https://github.com/komutan/NMeCab/blob/master/README.md?plain=1#L70) の直後に、同じ構成（`NMeCab`前提）のF#サンプルコードを追加して、後は現状通り（いちいちF#のコードは追加しない）で構いません。多分、F#erはC#のコードを見れば理解できます :) `MeCab.Tagger` の辺りが違うと思うので、`libNMeCab`のリリースをした後で追記する、で構わないと思います。 また、`MeCab.DotNet` の説明には: * エンジンのコードがNMeCab同等になった * バージョンアップされたパッケージが提供されているが、将来はarchiveされる * インターフェイスは元と同等なので、`MeCab.DotNet` のリポジトリを参照の事 のように書こうと思います。補足があればそれも盛り込むので、もしあれば指摘ください。 以上は `NMeCab` 側のREADMEの話で、`MeCab.DotNet` の方は同時期に `NMeCab` への移行を促すようにREADMEに追記します。
@komutan ありがとうございます、私もこの件は急いでいる話でもないので、ゆっくりと進めることが出来れば良いかなと思っています。 そちらの都合が付きそうでしたら、改めてメンション頂ければと思います。 具体的な進行が開始されるまでは、このissueを保持しておきたいのですが良いでしょうか？
@komutan なるほど、案について理解しました。 現状のMeCab.DotNet側のシグネチャを、NMeCabベースの実装で再現が可能ということであれば、明らかに不要となりそうでも、一旦両者のインターフェイスを出来るだけ再現するのが良いと思います（一般的なリファクタリングの原則に従って）。 例えば、Parseの戻り値がNMeCabでリッチな方向に変更されているのはそういった理由があると思いますので、最終的にはMeCab.DotNetのstringを返すバージョンは廃止するように寄せたほうが良いと思っています。 戻り値の型だけが異なるオーバーロードは許されませんが、MeCab.DotNetとNMeCabでは名前空間が異なるので、案の方法で寄せるので良いと思います。部分解析についてもOKです。例外で検出できることと、アナウンスすれば良いでしょう。 （部分解析が必要なユーザーは、最終版を使い続ける選択肢を残せます） --- リポジトリの一本化ですが、 1. NMeCab(GitHubのここ)ベース 2. MeCab.DotNetベース 3. 両者を維持しつづける 4. その他 とあると思います。このうち3はそもそもこの作業をやる意味がなさそうなので無いと思っています（ずっと両者を維持し続けるという意味です）が、どうでしょうか？ 1や2の場合は、organizationを立ててそちらにリポジトリを移設し、しかる後にどちらかをアーカイブして片方に誘導する、ということにすれば、片方の履歴は継続され、片方はアーカイブで残るので良いと思います。 私的には、私の方の履歴はほぼ移植作業（NuGetのパッケージ作るところとかあったかもしれない）なので、実質的には参照さえできればアーカイブで問題ないかなと思っています（アーカイブでも参照したい人がいるかどうか疑わしいですが :) NMeCabの履歴を残す場合の案（NMeCabリポジトリを使う）: 1. NMeCabリポジトリ側でMeCab.DotNetの互換実装を行う 2. NMeCabリポジトリ側でMeCab.DotNetのパッケージ生成できるようにする 3. 融合されたパッケージのリリース 4. リポジトリを、organizationに移動 5....
@komutan 了解しました、ではNMeCabのリポジトリで作業を進行して、リポジトリの以降は次の段階でやりましょう。 開発速度については全く気にしていませんので（GitHubでやる上にOSSの活動なので、非同期進行で一向に構いませんよ :) 無理のない範囲でやって下さい。 とりあえず最初のそちらの変更分が出来たら、こちらでもそれを評価してみます。ブランチ切った時点でPR立てて、実際の修正はその上でやるのが良いかと思います。
@komutan > コーディングを終えたので、PR #34 をレビュー可能にしました。ご確認いただけたら幸いです。 すいません、reactしてたのに完全に見落としていました orz レビューします
Related #50
41cf424cffe5668d98fe0258ed1c012958b7f09f Monitor lock feature implemented 🎉
It's bit difference handling. * The constant instance (currently only System_String with generated IL2C_CONST_STRING() macro), they can't set any instance flag at the IL2C_REF_HEADER.characteristic. Because it'll place in the rdata...
We need to extract automated how much lack members on IL2C. I think it difficulty topic: * We can traverse all members on mscorlib/corlib by cecil and/or reflection. * Very...
I will read/check too nanoFramework.CoreLibrary, I feel will take some idea from it. It may be difficult to port nanoFramework.CoreLibrary as it is. (Because there is a difference between AOT...