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

SJIS-win などのエンコーディングが mb_list_encodings の一覧から消えているが、マニュアルには残っている #32

Open
mumumu opened this issue Jan 11, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@mumumu
Copy link
Member

mumumu commented Jan 11, 2022

refs: php/php-src@e245985

  • A) PHP 8.1 から、SJIS-win, SJIS-open, SJIS-ms は、 CP932 のエイリアスとして扱われるようになった
  • B) A) のエンコーディングが、mb_list_encodings の出力から消えた
  • C) だが、マニュアルには残っているので、修正すべき
    • migration guide も修正対象に含めるべきと考えられる
    • 但し、mbstring の関数でこれらのエンコーディングが使えなくなったわけではない
      • おそらくは mb_list_encodings の例だけをまずは修正することになると考えられる。

そもそも A) や B) が php-src 的に正しい修正かが判断できないので、まずは、日本語版の issue に記録しておく。
現状が追認されるようなら、英語版も含めたマニュアルそのものの修正も視野に入れる。

@mumumu mumumu added the bug Something isn't working label Jan 11, 2022
@mumumu
Copy link
Member Author

mumumu commented Apr 6, 2022

現状が追認されるようなら、英語版も含めたマニュアルそのものの修正も視野に入れる

現状は追認されず、issue が立ちました。下記がどう対応されるかを見守り、対応を考えます。

php/php-src#8308

@mumumu
Copy link
Member Author

mumumu commented Jun 15, 2022

[memo] 問題のあるマッピングは修正されたものの、mb_list_encodings はそのまま。

php/php-src#8719

@mumumu
Copy link
Member Author

mumumu commented Nov 14, 2022

php/php-src#8308 は解決され、8.2.0RC の時点で、SJIS-Win も復活しています。

https://3v4l.org/GH0Gj#v8.2rc3

@mumumu
Copy link
Member Author

mumumu commented Nov 14, 2022

まとめ

  • 8.1.0 から 8.1.7 まで、SJIS-Win は CP932 のエイリアスとして扱われ、mb_list_encondings から削除されていた。
  • issue (The return value of mb_list_encodings() doesn't contain 'SJIS-win' php-src#8308) が立ち、各位の努力によって 8.1.8 から SJIS-Win は復活した。
  • つまり、現状は追認されなかったので、その旨を Changelog に折りを見て記録したほうが良さそう。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant