今日はエンジン/一部拡張の不要コード削除、master のテストこけ修正、hash_init() でのパラメータチェック失敗時の不要なオブジェクト生成削除、VM スタック伸長でページサイズの指定が反映されるようにする修正、windows 版で 7.3.0 だと subst コマンド利用パスでのパスの扱いで互換性崩れていた問題の修正、pkg-config 利用の一層の推進、AppVeyor のバイナリ SDK バージョンを更新する修正があった!
2018-12-26
dstogov: Removed dead condition
dstogov: Fixed crashes introduced by 7e597f48e9fda982e930e4f617d2b2d98d8878a5 (only master was affected)
- https://github.com/php/php-src/commit/636b16402b7c086e51d80d709508391aa95ebed3
- 一昨日の修正 で master がクラッシュするようになった問題の修正
- 2954a30 のコミットコメントで指摘入ってたテストこけの奴かな
dstogov: Micro optimization
dstogov: Removed useless zval_ptr_dtor(return_value)
- https://github.com/php/php-src/commit/22c9d19144285efaa28ece5d510fbd9ab800ffa8
- ext/filter、ext/imap、ext/spl で、不要な zval_ptr_dtor(return_value) を削除
dstogov: Create object only after parameter checks
- https://github.com/php/php-src/commit/cbf642b7154d55ea6378d2c8dd5be2a57f995b88
- ext/hash で、hash_init() の際にパラメータチェックより前にオブジェクトの生成を行っていたのを修正
dstogov: Respect EG(vm_stack_page_size)
- https://github.com/php/php-src/commit/b45774eed047dad63bb3959df1e38de866346b78
- [7.3~]
- VM スタック拡張の際に EG(vm_stack_page_size) を反映するようにし、またページサイズを指定して初期化する Zend API zend_vm_stack_init_ex() を追加
- PR:3718 で swoole の中の人と話してた奴
- ↑のコメントで「適切なページサイズの指定どうしよう」で、「実際幾つかアプリ動かしてスタック伸長のログとってみるくらいじゃない?」となってるの、まあそうだよなあみたいな気持ち
weltling: Fixed bug #77317 DIR, FILE, realpath() reveal physical path for subst virtual drive
- https://github.com/php/php-src/commit/199914b42dccae1c87212dac988b14f57969652d
- [7.3~]
- windows で、7.3.0 では subst コマンドでドライブ文字を割り当てられたパスについて、DIR, FILE, realpath() が本来の物理パスを返してしまう問題の修正
eli-schwartz: ext/gd: use --with instead of --enable
- https://github.com/php/php-src/commit/285a077c1284544e938cd07cfa1fca3d56cf7726
- ext/gd のビルドで、
--enable-freetype
でなく--with-freetype
を指定するよう修正
eli-schwartz: Require pkg-config for external PCRE
- https://github.com/php/php-src/commit/c1a22f3d4e69cb87f8deeed7d03d9434e6901537
- ext/pcre のビルドで、バンドルされてるのでない PCRE を使う際、pkg-config を要求するよう修正
- ↑のとあわせて PR:3654 で話されてた奴
weltling: Update binary SDK version for AppVeyor
- https://github.com/php/php-src/commit/3c7dc7b4c4905c8999731d1af26987b5d5425754
- [7.2~]
- AppVeyor のバイナリ SDK バージョンを更新
dstogov: Removed transparent wrapper
- https://github.com/php/php-src/commit/9882929bfa4422765c1b1fde6a612fa48bfcce2e
- 不要な php_output_wrapper() を削除して php_output_write() を直接呼び出すよう修正