AuroraMiddleware/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't use %i format specifier for size_t values.

Browse Source 
In 64 bit build, size_t can be 64 bits while %i expects a 32 bit value.
Fix this by just avoiding the unnecessary use of size_t, the number of entries
in the index is not going to be greater than 2^32.

Closes #16163.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@76287 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin 2014-04-05 10:44:54 +00:00
parent eed86ca61a
commit 7238d2761f
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/html/helpwnd.cpp
View File

@ -1636,10 +1636,10 @@ void wxHtmlHelpWindow::DoIndexFind()
m_IndexList->Clear();
const wxHtmlHelpMergedIndex& index = *m_mergedIndex;
size_t cnt = index.size();
const unsigned cnt = index.size();
int displ = 0;
for (size_t i = 0; i < cnt; i++)
for (unsigned i = 0; i < cnt; i++)
{
if (index[i].name.Lower().find(sr) != wxString::npos)
{
@ -1689,7 +1689,7 @@ void wxHtmlHelpWindow::DoIndexFind()
}
wxString cnttext;
cnttext.Printf(_("%i of %i"), displ, cnt);
cnttext.Printf(_("%i of %u"), displ, cnt);
m_IndexCountInfo->SetLabel(cnttext);
m_IndexText->SetSelection(0, sr.length());
@ -1708,10 +1708,10 @@ void wxHtmlHelpWindow::DoIndexAll()
m_IndexList->Clear();
const wxHtmlHelpMergedIndex& index = *m_mergedIndex;
size_t cnt = index.size();
const unsigned cnt = index.size();
bool first = true;
for (size_t i = 0; i < cnt; i++)
for (unsigned i = 0; i < cnt; i++)
{
m_IndexList->Append(index[i].name, (char*)(&index[i]));
if (first)
@ -1727,7 +1727,7 @@ void wxHtmlHelpWindow::DoIndexAll()
}
wxString cnttext;
cnttext.Printf(_("%i of %i"), cnt, cnt);
cnttext.Printf(_("%u of %u"), cnt, cnt);
m_IndexCountInfo->SetLabel(cnttext);
}