-
- いいね！0
病気の美少女
@lyrical_logical
（Play2 の）こういうの、直し始めたらきりがないので、拒否られても仕方ないと思いますね…この手の trivial なやつは、単体で投げずにバグフィックスとかする前にリファクタリングした方がいいぞってなったときに、こっそりやってしまっておくといいと思います
2013-01-06 06:01:21
病気の美少女
@lyrical_logical
なんか昨年末にツイートオーしたばかりな気がしてあれだけど、野暮なこというと、この手の関数のアロケーションなんてどうせ jit でスタックでされることになるし、そうでなくても GC のこと考えなければアロケーション単体のコスト安いし、主張するなら map のほうかなあと思いました。
2013-01-06 06:22:13
Kenji Yoshida
@xuwei_k
@lyrical_logical 拒否られたこと自体というより、なんで拒否られたのか？とか、なにを考えて拒否したのか？が謎すぎるあたりが https://t.co/frhPFTPt この件は、パフォーマンス(大した問題ではないが)的にも、読みやすさ的にも明らかに無駄なはず・・・
2013-01-06 07:39:45
病気の美少女
@lyrical_logical
@xuwei_k アレーその辺は釈迦にセポーかなと思って書かなかったのですが、Play2 はオープンソースなフレームワークだけどなんかコードの所有権のために同意書とか必要になって面倒になってて、この手のプルリクは（お互いにとって）嬉しくないということだそうです
2013-01-06 07:45:21
病気の美少女
@lyrical_logical
@xuwei_k あとはこの手の trivial なプルリクに対しての姿勢ですよね。こういうの一度受け入れちゃうと、他のも受け入れるしかなくなるけど、それは一定面倒なことで…向こうは仕事でやってるわけですからね。メリット小さい仕事を増やすようなことを避けるのは、まあ妥当なのでは。
2013-01-06 07:49:38
病気の美少女
@lyrical_logical
@xuwei_k 本当のところは聞いてみないとわかりませんけどね。なんにせよ author と commiter の意識の差というのは往々にしてあるものなので、コミュニティというかプロジェクトというか、そういうのにあわせてうまいこと見せないとみたいな…一般論ですけど
2013-01-06 07:54:30
Kenji Yoshida
@xuwei_k
@lyrical_logical 似たような(?)こんなもの https://t.co/WC0fMWKe は取り入れられたことあるし、そのあたりの基準が謎ぃ。(自分はサインしたことあるので)mergeボタン押すだけの作業(?)がそれほどが面倒くさいのか？という疑問は残りますけども
2013-01-06 08:08:20
Kenji Yoshida
@xuwei_k
@lyrical_logical まぁたしかに(詳しい事情はしらないが)昔はこんなこともあったらしいし https://t.co/nPIlwzxk 明らかにそういう面はありそうですね・・・。しかしそういう態度だと、よけいに貢献したくなくなるし、使いたくなくなる・・・
2013-01-06 08:09:57
病気の美少女
@lyrical_logical
@xuwei_k こっちはまあコードの妥当性みたいなのがさっきのよりは大分ましというか元の酷すぎだったとかでは。その辺の匙加減はまあそれぞれでしょう。
2013-01-06 08:20:36
病気の美少女
@lyrical_logical
@xuwei_k ならまあ使わなければいいのでは、仕事なら諦めましょう…野暮なことを言いますが、よかれと思ってやったことを拒否されたからと悲観的感情的になりすぎていませんか。別に Contribution を特別求められてるわけでもないわけですし、あんまり気にしても仕方ないですよ
2013-01-06 08:21:57
Kenji Yoshida
@xuwei_k
@lyrical_logical 「よかれと思ってやったことを拒否されたからと悲観的感情」はまぁもちろんその通りなんですが、今後(主に仕事で?)関わっていくことになりそうなので、その感情をなにかプラスな方向に活かしたいけど、現状なにもできなくてイライラしてますね・・・
2013-01-06 08:29:09
病気の美少女
@lyrical_logical
@xuwei_k そこはまあ、さっきも言いましたけど Evolution 回りでなんか問題見つけたときに、今回の修正をこっそりといれておいて通ったらニヤニヤするのを目標にするとかで…（本当は関係のない差分こっそりいれるとかは良くないですけど）
2013-01-06 08:32:54
Toshiyuki Takahashi
@tototoshi
http://t.co/s8pBFvFz Evolutions はパフォーマンス関係ないしなあ。まあ単にめんどいなあと思われたんじゃないかな。
2013-01-06 10:24:06
病気の美少女
@lyrical_logical
@tototoshi というか対人メタ張ると guillaumebort さんならまあこういうのは蹴るだろうなという感じはしますね…
2013-01-06 10:26:50
病気の美少女
@lyrical_logical
@tototoshi うーんというか、諸々の都合とか除いても、この差分なら普通に「マージする必要なし」と判断しそうだなあと思います。几帳面さはどうでしょうね。コードからは几帳面さ感じられないけど、もともと Scala の人じゃないからコードからそういうの推し量ってもなあ…
2013-01-06 10:30:40
病気の美少女
@lyrical_logical
@tototoshi なんというかこのアロケーションの reduce と同程度にはこの差分も unnecesarry だと判断しそうだなあという
2013-01-06 10:31:31
病気の美少女
@lyrical_logical
@tototoshi あー…でも几帳面さというか丁寧さはコミットからは感じられませんね…割とでかいコミットとかごちゃまぜのコミットしてる印象が…まあ 1.0 リリースで忙しい時期のコミットの印象が強いだけかもしれませんが。
2013-01-06 10:33:49
Kenji Yoshida
@xuwei_k
@tototoshi @lyrical_logical (この件は特に関係なく) 全体的にPlay2のコードが他のScalaのプロダクトと比べて、几帳面さが足りなすぎじゃないですかね・・・
2013-01-06 10:33:55
Toshiyuki Takahashi
@tototoshi
@xuwei_k @lyrical_logical ぷるリクエストコピペで取り込むとかはさすがに最近はしてない(とおもいます)(だったらいいな)。あのころのコミット量はちょっと多すぎて真面目にマージするのめんどくさかっただろうなあって気はします。
2013-01-06 10:36:22
Kenji Yoshida
@xuwei_k
ただtwitterなどで文句だけ言うのはいや https://t.co/Fo2H84Ky だから、どうでもいいような細かいpull reqしてみたけど、もうちょっと空気読まないとダメみたいなので、ソーシャルコーディング難しい
2013-01-06 10:39:49