LoginSignup
1
0

More than 3 years have passed since last update.

php-master-changes 2019-05-20

Posted at

今日は *.re へ変えていたファイルの *.l への再リネーム、テストの修正があった!

2019-05-20

krakjoe: Revert "Rename *.l files to *.re"

  • https://github.com/php/php-src/commit/e11233dc492a96844d22e4a3ed8e9e2d8a36651c
  • [7.4~]
  • 先日の修正 をリバート
  • PR:4172 で地味に喧嘩しててふいた
  • 「*.l で分かるからわざわざ変えなくていいんじゃない?」と俺も最初思ったんだけど、よーく考えると本当はこれ petk の人、というか統一が本来正しい気もする(結局リバートされちゃったけど)
  • 分かってる人にとってはどっちでもよい奴で、分かってない新規の人の参入障壁を下げるため構造を単純化していく、という話であって、分かってる人に分かってない人の気持ちは分からないので、その取り組みにはシンプルな原則論(同じ種類のファイルは同じ種類に見えるよう命名するとか)へ立ち返っての対応が適切
  • 外部ドキュメントなんて必ず時代遅れになるもんだし、GitHub からのファイルのコミットログの見え方はともかく blame は見れるので、気にせず統一の方向に直しちゃってよかったんじゃないかな、みたいな(ただどちらかというと *.re は親しみないけど *.l は親しみあるので、もう *.l 統一でよかったんじゃないかな、みたいな)
  • まあ当事者たちも言ってるように、ぶっちゃけどうでもいい些細な話だけどね!

cmb69: Fix test wrt. opcache.error_log

1
0
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
1
0