Skip to content

Commit

Permalink
#88: C4477 の警告を修正する
Browse files Browse the repository at this point in the history
  • Loading branch information
m-tmatma committed Jun 10, 2018
1 parent 0159e92 commit e1c0717
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions sakura_core/_os/CClipboard.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -227,10 +227,10 @@ bool CClipboard::SetHtmlText(const CNativeW& cmemBUf)
size_t size = cmemUtf8.GetStringLength() + 134;
cmemHeader.AppendString("Version:0.9\r\n");
cmemHeader.AppendString("StartHTML:00000097\r\n");
sprintf( szFormat, "EndHTML:%08d\r\n", size + 36 );
sprintf( szFormat, "EndHTML:%08Id\r\n", size + 36 );

This comment has been minimized.

Copy link
@berryzplus

berryzplus Jun 11, 2018

Contributor

この書き方 %Id は Microsoft 独自仕様です。他コンパイラでは正しく動かない可能性があります。

MinGW64では「'I' という型指定子はない」という警告が出ますが、単に無視されるだけみたいでした。
結果として期待通り64bit整数が出力されるようなので、MinGW64に関しては影響なさそうです。

iPhoneですごい量の修正に見えたので帰宅してから見てます。
あらためて見直したらちょこっとだったので 😢 でした。

This comment has been minimized.

Copy link
@m-tmatma

m-tmatma Jun 13, 2018

Author Member

MinGW はgcc と互換ですか?
gcc だと 以下に情報がまとめられています。http://d.hatena.ne.jp/pyopyopyo/touch/20110530/p1

cmemHeader.AppendString(szFormat);
cmemHeader.AppendString("StartFragment:00000134\r\n");
sprintf( szFormat, "EndFragment:%08d\r\n", size );
sprintf( szFormat, "EndFragment:%08Id\r\n", size );
cmemHeader.AppendString(szFormat);
cmemHeader.AppendString("<html><body>\r\n<!--StartFragment -->\r\n");
CNativeA cmemFooter;
Expand Down
2 changes: 1 addition & 1 deletion sakura_core/util/shell.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -607,7 +607,7 @@ BOOL MyWinHelp(HWND hwndCaller, UINT uCommand, DWORD_PTR dwData)
dwData = 1; // 目次ページ

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 );
ShellExecute( ::GetActiveWindow(), NULL, buf, NULL, NULL, SW_SHOWNORMAL );
}

Expand Down

0 comments on commit e1c0717

Please sign in to comment.