use WXSIZEOF(x) instead of sizeof(x)/sizeof(x[0]) (closes #10701)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@61296 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2009-07-02 22:43:31 +00:00
parent 609aa390f4
commit 9611b7975f
6 changed files with 20 additions and 20 deletions

View File

@@ -934,7 +934,7 @@ void wxDateTime::GetAmPmStrings(wxString *am, wxString *pm)
// assert, even though it is a perfectly legal use. // assert, even though it is a perfectly legal use.
if ( am ) if ( am )
{ {
if (wxStrftime(buffer, sizeof(buffer)/sizeof(wxChar), _T("%p"), &tm) > 0) if (wxStrftime(buffer, WXSIZEOF(buffer), _T("%p"), &tm) > 0)
*am = wxString(buffer); *am = wxString(buffer);
else else
*am = wxString(); *am = wxString();
@@ -942,7 +942,7 @@ void wxDateTime::GetAmPmStrings(wxString *am, wxString *pm)
if ( pm ) if ( pm )
{ {
tm.tm_hour = 13; tm.tm_hour = 13;
if (wxStrftime(buffer, sizeof(buffer)/sizeof(wxChar), _T("%p"), &tm) > 0) if (wxStrftime(buffer, WXSIZEOF(buffer), _T("%p"), &tm) > 0)
*pm = wxString(buffer); *pm = wxString(buffer);
else else
*pm = wxString(); *pm = wxString();

View File

@@ -831,13 +831,13 @@ void wxDebugContext::OutputDumpLine(const wxChar *szFormat, ...)
int count; int count;
va_list argptr; va_list argptr;
va_start(argptr, szFormat); va_start(argptr, szFormat);
buf[sizeof(buf)/sizeof(wxChar)-1] = _T('\0'); buf[WXSIZEOF(buf)-1] = _T('\0');
// keep 3 bytes for a \r\n\0 // keep 3 bytes for a \r\n\0
count = wxVsnprintf(buf, sizeof(buf)/sizeof(wxChar)-3, szFormat, argptr); count = wxVsnprintf(buf, WXSIZEOF(buf)-3, szFormat, argptr);
if ( count < 0 ) if ( count < 0 )
count = sizeof(buf)/sizeof(wxChar)-3; count = WXSIZEOF(buf)-3;
buf[count]=_T('\r'); buf[count]=_T('\r');
buf[count+1]=_T('\n'); buf[count+1]=_T('\n');
buf[count+2]=_T('\0'); buf[count+2]=_T('\0');

View File

@@ -442,7 +442,7 @@ void wxFontRefData::MacFindFont()
kATSUQDCondensedTag , kATSUQDCondensedTag ,
kATSUQDExtendedTag , kATSUQDExtendedTag ,
}; };
ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ByteCount atsuSizes[WXSIZEOF(atsuTags)] =
{ {
sizeof( ATSUFontID ) , sizeof( ATSUFontID ) ,
sizeof( Fixed ) , sizeof( Fixed ) ,
@@ -460,7 +460,7 @@ void wxFontRefData::MacFindFont()
Fixed atsuSize = IntToFixed( m_info.m_pointSize ); Fixed atsuSize = IntToFixed( m_info.m_pointSize );
ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal; ATSUVerticalCharacterType kHorizontal = kATSUStronglyHorizontal;
FMFontStyle addQDStyle = m_info.m_atsuAdditionalQDStyles; FMFontStyle addQDStyle = m_info.m_atsuAdditionalQDStyles;
ATSUAttributeValuePtr atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ATSUAttributeValuePtr atsuValues[WXSIZEOF(atsuTags)] =
{ {
&m_info.m_atsuFontID , &m_info.m_atsuFontID ,
&atsuSize , &atsuSize ,
@@ -474,7 +474,7 @@ void wxFontRefData::MacFindFont()
status = ::ATSUSetAttributes( status = ::ATSUSetAttributes(
(ATSUStyle)m_macATSUStyle, (ATSUStyle)m_macATSUStyle,
sizeof(atsuTags) / sizeof(ATSUAttributeTag) , WXSIZEOF(atsuTags),
atsuTags, atsuSizes, atsuValues); atsuTags, atsuSizes, atsuValues);
wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") ); wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") );

View File

@@ -869,19 +869,19 @@ wxMacCoreGraphicsFontData::wxMacCoreGraphicsFontData(wxGraphicsRenderer* rendere
kATSUSizeTag , kATSUSizeTag ,
kATSUColorTag , kATSUColorTag ,
}; };
ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ByteCount atsuSizes[WXSIZEOF(atsuTags)] =
{ {
sizeof( Fixed ) , sizeof( Fixed ) ,
sizeof( RGBColor ) , sizeof( RGBColor ) ,
}; };
ATSUAttributeValuePtr atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ATSUAttributeValuePtr atsuValues[WXSIZEOF(atsuTags)] =
{ {
&atsuSize , &atsuSize ,
&atsuColor , &atsuColor ,
}; };
status = ::ATSUSetAttributes( status = ::ATSUSetAttributes(
m_macATSUIStyle, sizeof(atsuTags) / sizeof(ATSUAttributeTag) , m_macATSUIStyle, WXSIZEOF(atsuTags),
atsuTags, atsuSizes, atsuValues); atsuTags, atsuSizes, atsuValues);
wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") ); wxASSERT_MSG( status == noErr , wxT("couldn't modify ATSU style") );
@@ -2268,15 +2268,15 @@ void wxMacCoreGraphicsContext::DoDrawRotatedText(const wxString &str,
{ {
kATSULineRotationTag , kATSULineRotationTag ,
}; };
ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ByteCount atsuSizes[WXSIZEOF(atsuTags)] =
{ {
sizeof( Fixed ) , sizeof( Fixed ) ,
}; };
ATSUAttributeValuePtr atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ATSUAttributeValuePtr atsuValues[WXSIZEOF(atsuTags)] =
{ {
&atsuAngle , &atsuAngle ,
}; };
status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag), status = ::ATSUSetLayoutControls(atsuLayout , WXSIZEOF(atsuTags),
atsuTags, atsuSizes, atsuValues ); atsuTags, atsuSizes, atsuValues );
} }
@@ -2285,15 +2285,15 @@ void wxMacCoreGraphicsContext::DoDrawRotatedText(const wxString &str,
{ {
kATSUCGContextTag , kATSUCGContextTag ,
}; };
ByteCount atsuSizes[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ByteCount atsuSizes[WXSIZEOF(atsuTags)] =
{ {
sizeof( CGContextRef ) , sizeof( CGContextRef ) ,
}; };
ATSUAttributeValuePtr atsuValues[sizeof(atsuTags) / sizeof(ATSUAttributeTag)] = ATSUAttributeValuePtr atsuValues[WXSIZEOF(atsuTags)] =
{ {
&m_cgContext , &m_cgContext ,
}; };
status = ::ATSUSetLayoutControls(atsuLayout , sizeof(atsuTags) / sizeof(ATSUAttributeTag), status = ::ATSUSetLayoutControls(atsuLayout , WXSIZEOF(atsuTags),
atsuTags, atsuSizes, atsuValues ); atsuTags, atsuSizes, atsuValues );
} }

View File

@@ -893,7 +893,7 @@ void wxMacMLTEControl::AdjustCreationAttributes(const wxColour &background,
} ; } ;
err = TXNSetTypeAttributes( err = TXNSetTypeAttributes(
m_txn, sizeof(typeAttr) / sizeof(TXNTypeAttributes), m_txn, WXSIZEOF(typeAttr),
typeAttr, kTXNStartOffset, kTXNEndOffset ); typeAttr, kTXNStartOffset, kTXNEndOffset );
verify_noerr( err ); verify_noerr( err );

View File

@@ -515,7 +515,7 @@ bool wxSymbolPickerDialog::TransferDataToWindow()
{ {
// Insert items into subset combo // Insert items into subset combo
int i; int i;
for (i = 0; i < (int) (sizeof(g_UnicodeSubsetTable)/sizeof(g_UnicodeSubsetTable[0])); i++) for (i = 0; i < (int) WXSIZEOF(g_UnicodeSubsetTable); i++)
{ {
m_subsetCtrl->Append(g_UnicodeSubsetTable[i].m_name); m_subsetCtrl->Append(g_UnicodeSubsetTable[i].m_name);
} }
@@ -617,7 +617,7 @@ void wxSymbolPickerDialog::OnSymbolSelected( wxCommandEvent& event )
{ {
// Need to make the subset selection reflect the current symbol // Need to make the subset selection reflect the current symbol
int i; int i;
for (i = 0; i < (int) (sizeof(g_UnicodeSubsetTable)/sizeof(g_UnicodeSubsetTable[0])); i++) for (i = 0; i < (int) WXSIZEOF(g_UnicodeSubsetTable); i++)
{ {
if (sel >= g_UnicodeSubsetTable[i].m_low && sel <= g_UnicodeSubsetTable[i].m_high) if (sel >= g_UnicodeSubsetTable[i].m_low && sel <= g_UnicodeSubsetTable[i].m_high)
{ {