今日だけ今日だけと言いながらどのくらい続けられるだろうか
今たまたま時間とれるけど、より根性試されるのあまりとれなくなってからな気はする
2018-08-01
smalyshev: Clarify difference between security & regular bugfixes
- https://github.com/php/php-src/commit/201022c34791318034f37591d00a39fc00577654
- README.GIT-RULES のバグ修正の宛て方についての記述で、通常のバグ修正とセキュリティ修正の当たる先を区別
- PHP のバグ修正はサポート対象バージョンの中の下の方へ一旦適用し、そこから上のバージョンへマージしていくルール
- なおこの php-master-changes ではマージのコミットまでは見てない
smalyshev: Ugh, whitespace
smalyshev: Typo
laurence: Unused var
- https://github.com/php/php-src/commit/85145e509e6a779621d9fd8f99b0e16f3e26ca1f
- 不要な変数の除去
- 昨日の dmitory の「Improved method visibility checks」でコード整理がされた際に不要となった変数なもよう
- この行自体は5年前からあった行で、何で急に laurence がこれに気付いて直したのか、と最初思った
- PR 通さずに入る修正もなんとなく他の開発者が見てるみたいな
weltling: Fix clang compilation
- https://github.com/php/php-src/commit/36857ab52b8cafc91650ccc55053abe6b517f092
- clang でのコンパイル問題修正
- -fmodules オプション付きでコンパイルすると、intrinsics が全ロードされ、Visual Studio 由来のヘッダが読み込まれて宣言の衝突が起きるという話
- それで clang では intrin.h を読み込まないようにする、まではなんとなく話として分かったよーな気もするけれど、__zend_cpuid() が固定値詰めるだけの実装になるのは別によいのだろうか（よく分からず）
- -fmodules 自体は clang の Visual Studio 互換モードでのリンク問題を防ぐのに必要とか
2018-08-02
cmb69: Remove needless version checks
- https://github.com/php/php-src/commit/3d4fbdade43c040293e6de4a1b9fb4781c5c1ee5
- ext/intl で ICU の 不要なバージョンチェックを削除
- PHP 5.5.3 から configure 段階で ICU バージョン 4.0 以上を要求するようになったため、3.X 系のチェックは不要とのこと
weltling: Suppress more irrelevant clang warnings
- https://github.com/php/php-src/commit/d53978bd59d45c3e850418e91c2e9ebc11491bf3
- clang の警告を減らす用にコンパイルフラグの追加