diff --git a/common/widgets/bitmap_button.cpp b/common/widgets/bitmap_button.cpp index 0bc9c0c545..aaa28e0944 100644 --- a/common/widgets/bitmap_button.cpp +++ b/common/widgets/bitmap_button.cpp @@ -35,10 +35,13 @@ BITMAP_BUTTON::BITMAP_BUTTON( wxWindow* aParent, wxWindowID aId, const wxPoint& aPos, const wxSize& aSize, int aStyles ) : wxPanel( aParent, aId, aPos, aSize, aStyles ), - m_isRadioButton( false ), m_buttonState( 0 ), m_padding( 0 ), - m_acceptDraggedInClicks( false ), m_showBadge( false ), + m_isRadioButton( false ), + m_showBadge( false ), m_badgeColor( wxColor( 210, 0, 0, 0 ) ), // dark red - m_badgeTextColor( wxColor( wxT( "white" ) ) ) + m_badgeTextColor( wxColor( wxT( "white" ) ) ), + m_buttonState( 0 ), + m_padding( 0 ), + m_acceptDraggedInClicks( false ) { if( aSize == wxDefaultSize ) SetMinSize( wxButton::GetDefaultSize() + wxSize( m_padding * 2, m_padding * 2) ); @@ -53,6 +56,7 @@ BITMAP_BUTTON::BITMAP_BUTTON( wxWindow* aParent, wxWindowID aId, const wxBitmap& const wxPoint& aPos, const wxSize& aSize, int aStyles ) : wxPanel( aParent, aId, aPos, aSize, aStyles ), m_isRadioButton( false ), + m_showBadge( false ), m_buttonState( 0 ), m_padding( 5 ), m_acceptDraggedInClicks( false ) diff --git a/kicad/pcm/pcm_data.h b/kicad/pcm/pcm_data.h index c03c41ef53..7b9d0159a3 100644 --- a/kicad/pcm/pcm_data.h +++ b/kicad/pcm/pcm_data.h @@ -86,7 +86,7 @@ struct PACKAGE_VERSION // Not serialized fields std::tuple parsed_version; // Full version tuple for sorting - bool compatible; + bool compatible = true; }; diff --git a/pcbnew/tools/pcb_selection_tool.cpp b/pcbnew/tools/pcb_selection_tool.cpp index c8cceb9c05..f546bd1f26 100644 --- a/pcbnew/tools/pcb_selection_tool.cpp +++ b/pcbnew/tools/pcb_selection_tool.cpp @@ -1596,7 +1596,7 @@ int PCB_SELECTION_TOOL::selectSameSheet( const TOOL_EVENT& aEvent ) FOOTPRINT* footprint = dynamic_cast( item ); - if( footprint->GetPath().empty() ) + if( !footprint || footprint->GetPath().empty() ) return 0; ClearSelection( true /*quiet mode*/ );