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

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

Closed
m-tmatma opened this issue Jun 11, 2018 · 10 comments
Labels
x64 x64 対応

Comments

@m-tmatma
Copy link
Member

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

#89 (comment)

@m-tmatma m-tmatma added the x64 x64 対応 label Jun 11, 2018
@m-tmatma m-tmatma changed the title [x64対応] DWORD_PTR の変数の変数名に関して検討する [x64対応] (未着手) DWORD_PTR の変数の変数名に関して検討する Jun 11, 2018
@berryzplus
Copy link
Contributor

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

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

@m-tmatma m-tmatma changed the title [x64対応] (未着手) DWORD_PTR の変数の変数名に関して検討する [x64対応] (未着手) DWORD_PTR, INT_PTR, LONG_PTR の変数の変数名に関して検討する Jun 11, 2018
@m-tmatma
Copy link
Member Author

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

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

@m-tmatma
Copy link
Member Author

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

@kobake
Copy link
Member

kobake commented Jun 14, 2018

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

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

@kobake
Copy link
Member

kobake commented Jun 14, 2018

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

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

@berryzplus
Copy link
Contributor

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

@kobake
Copy link
Member

kobake commented Jun 23, 2018

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

@k-takata
Copy link
Member

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

@kobake
Copy link
Member

kobake commented Jun 23, 2018

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

これには同意です。

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

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

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

@berryzplus
Copy link
Contributor

一括対応は不要、の結論で閉じておきます。

必要なら新しくissue立ててください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x64 x64 対応
Projects
None yet
Development

No branches or pull requests

5 participants