GoDocでの全角文字使用時の見出し文字の制限について

[inatus]

GoDocで見出し文字として表示される条件は、どうやら以下のようです。

• 直前に改行文字が2つ続けて入っている
• 半角英字の大文字で始まっている

下記でも全角文字および半角英字の小文字で始まっている見出しは、通常のテキストとして表示されてしまいました。
http://godoc.org/github.com/gophersjp/go/src/cmd/go

対策のしようがないと思いますが、一応上げておきます。

[atotto]

そういえばそうでした。私もこれでわざと見出しを翻訳しなかったのを思い出しました。。
http://godoc.org/github.com/gophersjp/go/src/cmd/gc

日本語で戦うのであればここのメソッドを修正する必要がありそうですね。
http://golang.org/pkg/go/doc/#ToHTML
http://golang.org/src/pkg/go/doc/comment.go?s=6479:6541#L248

[inatus]

修正してみました。
こんな感じでしょうか。

[atotto]

ローカルで試してみました。いい感じですね！

ただ、もともとのテスト（comment_test.go）の中身が消えちゃってるので、
src/pkg/go/doc/comment_test.go を src/pkg/go/doc/comment_japanese_test.go とするか、マージしたほうが良いと思います。
もともとのテストも同時に走らせましたがパスしたので問題なさそうです。

これって、goの本家に反映したりしないですか？

[inatus]

確認ありがとうございます！

ただ、もともとのテスト（comment_test.go）の中身が消えちゃってるので、
src/pkg/go/doc/comment_test.go を src/pkg/go/doc/comment_japanese_test.go とするか、マージしたほうが良いと思います。

すみません、ちゃんと見てませんでした。。。
comment_japanese_test.go に変更します。

これって、goの本家に反映したりしないですか？

なんらかの形で提案したいところですね。
コード（nonlatincharactersupport.go）を見ていただくとわかるのですが、
これだと言語が増えれば増えるほどパフォーマンスが落ちます。
doc.goのどこかに言語情報を埋め込む（たとえばドキュメントの冒頭に「// language: ja」と書く）などして、
その言語の判定だけをさせるようにしたほうがよいかなあ、とも思っています。
どうですかね。

[atotto]

たしかにパフォーマンスに影響しますね。
言語が増えることを想定すると言語情報を埋め込んでしまうのがいい気がします。

[inatus]

とりあえずできました。
コメントに1行目に
{"language": "ja"}
とJSONで書くと日本語のチェックがオンになります。

で、これ本家にあげるにはどうしたらいいんでしょうか。
Issueであげて既存ソースのパッチと追加のソースファイルを添付すればいいのでしょうか。。。

[atotto]

この場合どのルートがいいのか自分も不案内ですが、issueかgolang-nutsのメーリングリストですかねぇ。

ちなみに、ソースはcodereviewにアップしておけば手作業でパッチつくらなくて良いです。

コミット（codereview）に関する流れは http://godocjp.herokuapp.com/doc/contribute.html の通りですが、大まかな流れを書くと：

• goのリポジトリをチェックアウト
• hgにcodereviewの拡張設定を書く
• 修正する
• hg changeする
• issue or メーリングリスト(golang-nuts)に投げる （hg mailはしない）

最後のissue or メーリングリストは最初に言っておくか最後にcodereviewのリンクまで貼って言うかはおまかせします(^^ゞ

[inatus]

あ、すみません、またちゃんと見てませんでした。
まずはメーリングリストで議論しろと書いてありますね。
とりあえず、トピックを上げてみます。

[inatus]

メーリスに投稿しました。
が、初投稿にはモデレータの認証が必要らしく、表示されるのに2，3時間かかりますと書いてあるのにまる一日経った今でもまだ表示されていません...
まだ休日なのでアメリカ時間の週明けまで待ってみます。

[inatus]

掲載されました。

https://groups.google.com/forum/#!topic/golang-nuts/lsMbGtl9e7k

が、なんかもうすでに埋もれてるんですが...

[atotto]

お、コメントついてますね。

[inatus]

issueをあげろとのことですね。
実現方式についてはissueの中で議論すればよいということですかね。
golang-nutsに書いたのと同様の内容であげときます。

[inatus]

issueあげました。

https://code.google.com/p/go/issues/detail?id=7349

[inatus]

外部の人からいくつかコメントがついてますね。
難クセつけてくる輩がいたのでうるせーと返しときました。

こんな感じで議論続けていったらそのうち中の人がacceptedにしてくれるんですかね...

[atotto]

さすがですｗ
長期戦になりそうですねぇ。
ひとまず今公開してるところにパッチするようにしましょうかね。。

[inatus]

そうですね！
いまのところブランチ切ってソース置いてるんですが、masterにマージしたほうがいいですか？

https://github.com/gophersjp/go/tree/issue/15

[inatus]

ちょっと動きがありましたね。
golangの中の人からコメントがありました。
go.textレポジトリのパッケージを使って実現したいとのことで、HelpWantedのステータスになりました。
とりあえずgo.textの中の人が降臨するのを待ちわびてればいいんですかね...

https://code.google.com/p/go/issues/detail?id=7349

[atotto]

ほぅ、 go.text https://code.google.com/p/go/source/browse/?repo=text ですか。
どんなふうになるんだろう・・

いま切ってあるブランチですが、ちょっとお待ちください。
ひとまず、drone.ioからherokuに渡すときにどうしたらうまく行くかを考えます。。

[inatus]

はい、encodingの中にjapaneseがありますけど、こういうのを使うんでしょうか。。。
降臨を待ちましょう。
ラインナップが日本語、韓国語、簡体字、繁体字だけなのがなんとも興味深いです。

ブランチについては了解しました。