Sakura: [x64対応] (未着手) DWORD_PTR, INT_PTR, LONG_PTR の変数の変数名に関して検討する

Created on 11 Jun 2018  ·  9Comments  ·  Source: sakura-editor/sakura

[x64対応] DWORD_PTR の変数の変数名に関して検討する

https://github.com/sakura-editor/sakura/pull/89#discussion_r194280548

x64

All 9 comments

実施するなら _PTR 系全部を対象にご検討いただけませんか?

INT_PTR, LONG_PTRなど。全部x64化で32bit⇒64bitになると思いますので。

実施するなら _PTR 系全部を対象にご検討いただけませんか?

タイトル変えました。
PR は分けたほうがいいかもしれませんが。

C:\Program Files (x86)\Windows Kits10\Include 以下にある標準のヘッダは
特に DWORD_PTR とわかるような変数名つけてないですね。
INT_PTR や LONG_PTR も同様

C:\Program Files (x86)\Windows Kits10\Include 以下にある標準のヘッダは
特に DWORD_PTR とわかるような変数名つけてないですね。
INT_PTR や LONG_PTR も同様

確かに……。これめちゃくちゃ気持ち悪いですね……。

https://github.com/search?o=desc&p=4&q=DWORD_PTR&s=indexed&type=Code

ざっくり GitHub 上のコードを眺めた感じだとプリフィックスを付けないケースが多いように感じました。

この件ってどうなってるんでしたっけ?
気にしなくていい、って結論にする認識でOK?

結論はまだ出していないという認識です。他にもっと大事な議論があるので自分はそっち優先してます。

ハンガリアン記法はもはや下火なのでは?
***_PTR 用の公式なプレフィックスは存在しないと思います。

ハンガリアン記法はもはや下火なのでは?

これには同意です。

ただ、ハンガリアン記法なコードが残っていることについては自分としては直しても放置でもどっちでも良いと思っています。

ただ、DWORD_PTR dwData; みたいな宣言はちょっと見逃せないなーって思ってます。これじゃ宣言部見ずに利用部だけ見ると DWORD 型だと勘違いされてしまいそうなので。(https://github.com/sakura-editor/sakura/pull/89#discussion_r194280548 で指摘したところ)

もしかして自分が知らないだけで DWORD_PTR 型の変数のプリフィックスを dw にするような慣習ってあったりするんですかね。

Was this page helpful?
0 / 5 - 0 ratings