Fix a few coverity errors
This commit is contained in:
parent
f364e81a91
commit
ec59361f75
|
@ -52,7 +52,8 @@ KIDIALOG::KIDIALOG( wxWindow* aParent, const wxString& aMessage,
|
|||
|
||||
KIDIALOG::KIDIALOG( wxWindow* aParent, const wxString& aMessage,
|
||||
KD_TYPE aType, const wxString& aCaption )
|
||||
: wxRichMessageDialog( aParent, aMessage, getCaption( aType, aCaption ), getStyle( aType ) )
|
||||
: wxRichMessageDialog( aParent, aMessage, getCaption( aType, aCaption ), getStyle( aType ) ),
|
||||
m_hash( 0 )
|
||||
{
|
||||
}
|
||||
|
||||
|
|
|
@ -45,6 +45,7 @@ BLOCK_SELECTOR::BLOCK_SELECTOR() :
|
|||
m_state = STATE_NO_BLOCK; // State (enum BLOCK_STATE_T) of block.
|
||||
m_command = BLOCK_IDLE; // Type (enum BLOCK_COMMAND_T) of operation.
|
||||
m_color = BROWN;
|
||||
m_appendUndo = false;
|
||||
}
|
||||
|
||||
|
||||
|
|
|
@ -48,8 +48,9 @@ static const char illegalFileNameChars[] = "\\/:\"<>|";
|
|||
*/
|
||||
wxString EscapeString( const wxString& aSource )
|
||||
{
|
||||
#if 1
|
||||
return aSource;
|
||||
|
||||
#else
|
||||
wxString converted;
|
||||
|
||||
for( wxUniChar c: aSource )
|
||||
|
@ -87,13 +88,15 @@ wxString EscapeString( const wxString& aSource )
|
|||
}
|
||||
|
||||
return converted;
|
||||
#endif
|
||||
}
|
||||
|
||||
|
||||
wxString UnescapeString( const wxString& aSource )
|
||||
{
|
||||
#if 1
|
||||
return aSource;
|
||||
|
||||
#else
|
||||
wxString converted = aSource;
|
||||
|
||||
converted.Replace( """, "\"" );
|
||||
|
@ -114,6 +117,7 @@ wxString UnescapeString( const wxString& aSource )
|
|||
converted.Replace( "&", "&" );
|
||||
|
||||
return converted;
|
||||
#endif
|
||||
}
|
||||
|
||||
|
||||
|
|
|
@ -87,6 +87,7 @@ AR_AUTOPLACER::AR_AUTOPLACER( BOARD* aBoard )
|
|||
m_gridSize = Millimeter2iu( STEP_AR_MM );
|
||||
m_progressReporter = nullptr;
|
||||
m_refreshCallback = nullptr;
|
||||
m_minCost = 0.0;
|
||||
}
|
||||
|
||||
|
||||
|
|
|
@ -197,6 +197,7 @@ public:
|
|||
long style = 0, const wxString choices[] = NULL, int unused = 0)
|
||||
: wxHtmlListBox( parent, id, pos, size, style )
|
||||
{
|
||||
m_units = MILLIMETRES;
|
||||
m_list = 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -78,8 +78,7 @@ void GRID_CELL_LAYER_RENDERER::Draw( wxGrid& aGrid, wxGridCellAttr& aAttr, wxDC&
|
|||
|
||||
|
||||
GRID_CELL_LAYER_SELECTOR::GRID_CELL_LAYER_SELECTOR( PCB_BASE_FRAME* aFrame, LSET aMask ) :
|
||||
m_frame( aFrame ),
|
||||
m_mask( aMask )
|
||||
m_frame( aFrame ), m_mask( aMask ), m_value( 0 )
|
||||
{
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue