-
Notifications
You must be signed in to change notification settings - Fork 167
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
Comments
実施するなら _PTR 系全部を対象にご検討いただけませんか? INT_PTR, LONG_PTRなど。全部x64化で32bit⇒64bitになると思いますので。 |
タイトル変えました。 |
C:\Program Files (x86)\Windows Kits\10\Include 以下にある標準のヘッダは |
確かに……。これめちゃくちゃ気持ち悪いですね……。 |
https://github.com/search?o=desc&p=4&q=DWORD_PTR&s=indexed&type=Code ざっくり GitHub 上のコードを眺めた感じだとプリフィックスを付けないケースが多いように感じました。 |
この件ってどうなってるんでしたっけ? |
結論はまだ出していないという認識です。他にもっと大事な議論があるので自分はそっち優先してます。 |
ハンガリアン記法はもはや下火なのでは? |
これには同意です。 ただ、ハンガリアン記法なコードが残っていることについては自分としては直しても放置でもどっちでも良いと思っています。 ただ、 もしかして自分が知らないだけで DWORD_PTR 型の変数のプリフィックスを |
一括対応は不要、の結論で閉じておきます。 必要なら新しくissue立ててください。 |
[x64対応] DWORD_PTR の変数の変数名に関して検討する
#89 (comment)
The text was updated successfully, but these errors were encountered: