Commit 35ddad21 authored by Dick Hollenbeck's avatar Dick Hollenbeck

wxWidgets Debug builds apparently have some tricky technique to validate

the match between wxString::Format() or wxString::Printf() format strings
with passed arguments, but does this at runtime, not compile time.
Fix some mismatches.  size_t is a 64 bit type on x86_64 whereas int and
unsigned are 32 bit types.  On 32 bit machines they are all 32 bits so
this error is probably not triggered.
parent 4082e122
...@@ -573,7 +573,7 @@ void SCH_EDIT_FRAME::LoadSettings() ...@@ -573,7 +573,7 @@ void SCH_EDIT_FRAME::LoadSettings()
m_findReplaceData->SetReplaceString( cfg->Read( ReplaceStringEntry, wxEmptyString ) ); m_findReplaceData->SetReplaceString( cfg->Read( ReplaceStringEntry, wxEmptyString ) );
// Load the find and replace string history list. // Load the find and replace string history list.
for ( size_t i = 0; i < FR_HISTORY_LIST_CNT; i++ ) for( int i = 0; i < FR_HISTORY_LIST_CNT; ++i )
{ {
wxString tmpHistory; wxString tmpHistory;
wxString entry; wxString entry;
...@@ -652,17 +652,17 @@ void SCH_EDIT_FRAME::SaveSettings() ...@@ -652,17 +652,17 @@ void SCH_EDIT_FRAME::SaveSettings()
cfg->Write( ReplaceStringEntry, m_findReplaceData->GetReplaceString() ); cfg->Write( ReplaceStringEntry, m_findReplaceData->GetReplaceString() );
// Save the find and replace string history list. // Save the find and replace string history list.
size_t i; unsigned i;
wxString tmpHistory; wxString tmpHistory;
wxString entry; // invoke constructor outside of any loops wxString entry; // invoke constructor outside of any loops
for ( i = 0; i < m_findStringHistoryList.GetCount() && i < FR_HISTORY_LIST_CNT; i++ ) for( i = 0; i < m_findStringHistoryList.GetCount() && i < FR_HISTORY_LIST_CNT; i++ )
{ {
entry.Printf( FindStringHistoryEntry, i ); entry.Printf( FindStringHistoryEntry, i );
cfg->Write( entry, m_findStringHistoryList[ i ] ); cfg->Write( entry, m_findStringHistoryList[ i ] );
} }
for ( i = 0; i < m_replaceStringHistoryList.GetCount() && i < FR_HISTORY_LIST_CNT; i++ ) for( i = 0; i < m_replaceStringHistoryList.GetCount() && i < FR_HISTORY_LIST_CNT; i++ )
{ {
entry.Printf( ReplaceStringHistoryEntry, i ); entry.Printf( ReplaceStringHistoryEntry, i );
cfg->Write( entry, m_replaceStringHistoryList[ i ] ); cfg->Write( entry, m_replaceStringHistoryList[ i ] );
......
...@@ -701,10 +701,11 @@ bool DRC::checkClearancePadToPad( D_PAD* aRefPad, D_PAD* aPad ) ...@@ -701,10 +701,11 @@ bool DRC::checkClearancePadToPad( D_PAD* aRefPad, D_PAD* aPad )
if( !trapezoid2trapezoidDRC( polyref, polycompare, dist_min ) ) if( !trapezoid2trapezoidDRC( polyref, polycompare, dist_min ) )
diag = false; diag = false;
} }
else // Should not occurs, because aPad and aRefPad are swapped else
// to have only aPad shape RECT or TRAP and aRefPad shape TRAP or RECT.
{ {
wxLogDebug( wxT( "unexpected pad shape" ) ); // Should not occur, because aPad and aRefPad are swapped
// to have only aPad shape RECT or TRAP and aRefPad shape TRAP or RECT.
wxLogDebug( wxT( "unexpected pad shape %d") , aPad->GetShape() );
} }
break; break;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment