Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

to be a gem #1

Closed
wants to merge 1 commit into from
Closed

to be a gem #1

wants to merge 1 commit into from

Conversation

masarakki
Copy link

No description provided.

@tadd
Copy link
Owner

tadd commented Sep 21, 2013

日本語のためのものなので日本語で。(って良くないでしょうか…)

PRありがとうございます。いくつかの部分に分けて、これからほとんどを取り込まさせていただこうと思います。

tadd added a commit that referenced this pull request Sep 21, 2013
tadd added a commit that referenced this pull request Sep 21, 2013
tadd added a commit that referenced this pull request Sep 21, 2013
tadd added a commit that referenced this pull request Sep 21, 2013
proposed by @masarakki at #1.
@tadd tadd mentioned this pull request Sep 21, 2013
@tadd
Copy link
Owner

tadd commented Sep 21, 2013

ひと通り取り込ませていただきました。

  • Gemfile.lockを無視
  • クラス名の変更
    • c52ab1d で修正しました。
    • 色々悩んでわざと長ったらしい名前にしてたんですが、悩みながらベストを探すのがいいですね。
    • なんとなく https://github.com/seejee/braille_ueb に合わせる形で braille-jaで切ってあります
    • というわけでリポジトリ名も変わっています
  • RSpec 導入
    • ごもっともです。 75e0f33 で入れました
  • Gem化
    • こちらもごもっとも。 65e4aa1 で修正しました
    • specと共に、早めにやろうと思っていたのでありがたかったです。

ただ以下の部分については、Rejectもしくは保留にしてあります。

  • ライセンス変更
    • これはRejectで! GNUラブなんです
    • 悩んでいるところもあるのですが、コピーレフトのライセンスを非コピーレフトにするより、その逆は難しいと考えているので、保守的な方を取っているという面もあります。
  • String#to_tenji
    • これは保留です。簡単なインターフェースが必要というのはそのとおりだと思いますが、一筋縄には行かないのです。詳しくは 簡単な使い方がない #2 で。
    • あとtenjiというのは、調べたのですが国際語ではないようなので、たぶんメソッドのシグネチャとしてはやっぱりbrailleにすると思っています。SELECT daimei FROM hondana とかみたいになって、微妙なので…。
    • 既存クラスの拡張の仕方はActiveSupportとかに習った方法なのでしょうか。大いに参考にさせていただきます。

というわけで長くなりましたが、 @masarakki さんからPRが飛んでくるとは思わなかったので、とてもうれしかったです 😄
ありがとうございました、一旦Closeします。何かあったら reopen か他の Issue をお待ちしてます。

@tadd tadd closed this Sep 21, 2013
@masarakki
Copy link
Author

点字ってわかんないんだけど日本語と他言語で同じ点字が違う文字に割り当てられたりしてるのかなぁ
とか色々考えてたら日本人が日本語を点字化するんだからto_tenjiのほうが逆にいいんじゃないかと思ってそうした
点字文化がよくわからないからなんとも言えない
文字コードになってるんだからある程度世界共通のポッチなんだろうとは思うけど

@tadd
Copy link
Owner

tadd commented Sep 21, 2013

素早いw コメントありがとうございます。なるほど、それは一理ありますね。自分もほとんど同じ立場の素人で、よく分かってないんです。

他の言語とコンフリクトするエンコード規則があって、ボッチ感が確信できたら、 to_tenji 入れてみるかもしれません。

@tadd
Copy link
Owner

tadd commented Sep 21, 2013

いまwikipediaを見比べた限りなのですが、

http://ja.wikipedia.org/wiki/%E7%82%B9%E5%AD%97#.E6.95.B0.E5.AD.97
http://en.wikipedia.org/wiki/Braille

数字・アルファベットについては >完全に一致< ですね。つまりかな部分の符号化は少なくとも英語とは被らないので、日本語点字にはある程度の上位互換性があるということに見えます。

ということで、あんまりボッチでないと仮定してbrailleだけで突っ切る方向で、とりあえず進んでみようと思います。実際のユーザーに会ってちょっと話したりもしてみます。

tadd added a commit that referenced this pull request Sep 21, 2013
encoding method of Japanese Braille seems to have some
compatibility with English's one.
this conclusion is derived by a discussion with @masarakki at #1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants