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対応] c4477 の警告を修正 #89

Merged
merged 1 commit into from
Jun 11, 2018

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma changed the title c4477 の警告を修正 [x64対応] c4477 の警告を修正 Jun 10, 2018
@m-tmatma m-tmatma self-assigned this Jun 10, 2018
@m-tmatma m-tmatma added the x64 x64 対応 label Jun 10, 2018
@@ -607,7 +607,7 @@ BOOL MyWinHelp(HWND hwndCaller, UINT uCommand, DWORD_PTR dwData)
dwData = 1; // �ڎ��y�[�W

TCHAR buf[256];
_stprintf( buf, _T("http://sakura-editor.sourceforge.net/cgi-bin/hid2.cgi?%d"), dwData );
_stprintf( buf, _T("http://sakura-editor.sourceforge.net/cgi-bin/hid2.cgi?%Id"), dwData );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正の本質ではないのですが、dwData という変数名が気になりました。
これを機に dwptrData のような名前に変更するのはどうでしょうか。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#93 に登録しました。

Copy link
Member

@kobake kobake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kobake kobake merged commit fb2a01f into sakura-editor:x64 Jun 11, 2018
@m-tmatma m-tmatma deleted the feature/fix-C4477-warning branch June 11, 2018 02:59
@m-tmatma m-tmatma added this to the next release milestone Jun 11, 2018
@ds14050 ds14050 added the x64 x64 対応 label Sep 18, 2018
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…arning

[x64対応] c4477 の警告を修正
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

Successfully merging this pull request may close these issues.

4 participants