realAscot/kicad-source
1
0
Fork 0
mirror of https://gitlab.com/kicad/code/kicad.git synced 2025-09-14 02:03:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

Don't store m_severities. It won't follow save control state.

Browse Source
This commit is contained in:
Jeff Young 2025-08-09 18:59:24 +01:00
parent 5805f67459
commit a45f8bc2f7
10 changed files with 59 additions and 188 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

143
common/widgets/wx_html_report_panel.cpp
View File

@ -37,22 +37,18 @@
#include <kiway_holder.h>
#include <project.h>
WX_HTML_REPORT_PANEL::WX_HTML_REPORT_PANEL( wxWindow* parent, wxWindowID id, const wxPoint& pos,
const wxSize& size, long style ) :
WX_HTML_REPORT_PANEL::WX_HTML_REPORT_PANEL( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size,
long style ) :
WX_HTML_REPORT_PANEL_BASE( parent, id, pos, size, style ),
m_reporter( this ),
m_severities( -1 ),
m_lazyUpdate( false )
{
syncCheckboxes();
m_htmlView->SetFont( KIUI::GetInfoFont( m_htmlView ) );
Flush();
Connect( wxEVT_COMMAND_MENU_SELECTED, wxMenuEventHandler( WX_HTML_REPORT_PANEL::onMenuEvent ),
nullptr, this );
Connect( wxEVT_COMMAND_MENU_SELECTED, wxMenuEventHandler( WX_HTML_REPORT_PANEL::onMenuEvent ), nullptr, this );
m_htmlView->Bind( wxEVT_SYS_COLOUR_CHANGED,
wxSysColourChangedEventHandler( WX_HTML_REPORT_PANEL::onThemeChanged ),
m_htmlView->Bind( wxEVT_SYS_COLOUR_CHANGED, wxSysColourChangedEventHandler( WX_HTML_REPORT_PANEL::onThemeChanged ),
this );
}
@ -83,8 +79,7 @@ REPORTER& WX_HTML_REPORT_PANEL::Reporter()
}
void WX_HTML_REPORT_PANEL::Report( const wxString& aText, SEVERITY aSeverity,
REPORTER::LOCATION aLocation )
void WX_HTML_REPORT_PANEL::Report( const wxString& aText, SEVERITY aSeverity, REPORTER::LOCATION aLocation )
{
REPORT_LINE line;
line.message = aText;
@ -105,12 +100,6 @@ void WX_HTML_REPORT_PANEL::Report( const wxString& aText, SEVERITY aSeverity,
}
void WX_HTML_REPORT_PANEL::SetLazyUpdate( bool aLazyUpdate )
{
m_lazyUpdate = aLazyUpdate;
}
void WX_HTML_REPORT_PANEL::Flush( bool aSort )
{
wxString html;
@ -181,7 +170,7 @@ wxString WX_HTML_REPORT_PANEL::generateHtml( const REPORT_LINE& aLine )
{
wxString retv;
if( !( m_severities & aLine.severity ) )
if( !( GetVisibleSeverities() & aLine.severity ) )
return retv;
if( KIPLATFORM::UI::IsDarkTheme() )
@ -281,73 +270,16 @@ static int RPT_SEVERITY_ALL = RPT_SEVERITY_WARNING | RPT_SEVERITY_ERROR | RPT_SE
RPT_SEVERITY_ACTION;
void WX_HTML_REPORT_PANEL::onCheckBoxShowAll( wxCommandEvent& event )
void WX_HTML_REPORT_PANEL::onCheckBox( wxCommandEvent& event )
{
if( event.IsChecked() )
m_severities = RPT_SEVERITY_ALL;
else
m_severities = RPT_SEVERITY_ERROR;
CallAfter(
[&]()
{
m_checkBoxShowAll->SetValue( GetVisibleSeverities() == RPT_SEVERITY_ALL );
} );
syncCheckboxes();
Flush( true );
}
void WX_HTML_REPORT_PANEL::syncCheckboxes()
{
m_checkBoxShowAll->SetValue( m_severities == RPT_SEVERITY_ALL );
m_checkBoxShowWarnings->SetValue( m_severities & RPT_SEVERITY_WARNING );
m_checkBoxShowErrors->SetValue( m_severities & RPT_SEVERITY_ERROR );
m_checkBoxShowInfos->SetValue( m_severities & RPT_SEVERITY_INFO );
m_checkBoxShowActions->SetValue( m_severities & RPT_SEVERITY_ACTION );
}
void WX_HTML_REPORT_PANEL::onCheckBoxShowWarnings( wxCommandEvent& event )
{
if( event.IsChecked() )
m_severities |= RPT_SEVERITY_WARNING;
else
m_severities &= ~RPT_SEVERITY_WARNING;
syncCheckboxes();
Flush( true );
}
void WX_HTML_REPORT_PANEL::onCheckBoxShowErrors( wxCommandEvent& event )
{
if( event.IsChecked() )
m_severities |= RPT_SEVERITY_ERROR;
else
m_severities &= ~RPT_SEVERITY_ERROR;
syncCheckboxes();
Flush( true );
}
void WX_HTML_REPORT_PANEL::onCheckBoxShowInfos( wxCommandEvent& event )
{
if( event.IsChecked() )
m_severities |= RPT_SEVERITY_INFO;
else
m_severities &= ~RPT_SEVERITY_INFO;
syncCheckboxes();
Flush( true );
}
void WX_HTML_REPORT_PANEL::onCheckBoxShowActions( wxCommandEvent& event )
{
if( event.IsChecked() )
m_severities |= RPT_SEVERITY_ACTION;
else
m_severities &= ~RPT_SEVERITY_ACTION;
syncCheckboxes();
Flush( true );
event.Skip();
}
@ -384,12 +316,8 @@ void WX_HTML_REPORT_PANEL::onBtnSaveToFile( wxCommandEvent& event )
if( !f.IsOpened() )
{
wxString msg;
msg.Printf( _( "Cannot write report to file '%s'." ),
fn.GetFullPath().GetData() );
wxMessageBox( msg, _( "File save error" ), wxOK | wxICON_ERROR,
wxGetTopLevelParent( this ) );
wxMessageBox( wxString::Format( _( "Cannot write report to file '%s'." ), fn.GetFullPath().GetData() ),
_( "File save error" ), wxOK | wxICON_ERROR, wxGetTopLevelParent( this ) );
return;
}
@ -423,44 +351,23 @@ void WX_HTML_REPORT_PANEL::SetLabel( const wxString& aLabel )
}
void WX_HTML_REPORT_PANEL::SetVisibleSeverities( int aSeverities )
{
if( aSeverities < 0 )
m_severities = RPT_SEVERITY_ALL;
else
m_severities = aSeverities;
syncCheckboxes();
}
int WX_HTML_REPORT_PANEL::GetVisibleSeverities() const
{
return m_severities;
}
int severities = 0;
if( m_checkBoxShowErrors->GetValue() )
severities |= RPT_SEVERITY_ERROR;
void WX_HTML_REPORT_PANEL::SetFileName( const wxString& aReportFileName )
{
m_reportFileName = aReportFileName;
}
if( m_checkBoxShowWarnings->GetValue() )
severities |= RPT_SEVERITY_WARNING;
if( m_checkBoxShowActions->GetValue() )
severities |= RPT_SEVERITY_ACTION;
wxString& WX_HTML_REPORT_PANEL::GetFileName( void )
{
return ( m_reportFileName );
}
if( m_checkBoxShowInfos->GetValue() )
severities |= RPT_SEVERITY_INFO;
void WX_HTML_REPORT_PANEL::SetShowSeverity( SEVERITY aSeverity, bool aValue )
{
switch( aSeverity )
{
case RPT_SEVERITY_INFO: m_checkBoxShowInfos->SetValue( aValue ); break;
case RPT_SEVERITY_ACTION: m_checkBoxShowActions->SetValue( aValue ); break;
case RPT_SEVERITY_WARNING: m_checkBoxShowWarnings->SetValue( aValue ); break;
default: m_checkBoxShowErrors->SetValue( aValue ); break;
}
return severities;
}

45
common/widgets/wx_html_report_panel.h
View File

@ -19,8 +19,7 @@
* with this program. If not, see <http://www.gnu.org/licenses/>.
*/
#ifndef __WX_HTML_REPORT_PANEL_H__
#define __WX_HTML_REPORT_PANEL_H__
#pragma once
#include <reporter.h>
#include <vector>
@ -43,12 +42,8 @@ public:
virtual ~WX_HTML_PANEL_REPORTER() {}
REPORTER& Report( const wxString& aText, SEVERITY aSeverity = RPT_SEVERITY_UNDEFINED ) override;
REPORTER& ReportTail( const wxString& aText,
SEVERITY aSeverity = RPT_SEVERITY_UNDEFINED ) override;
REPORTER& ReportHead( const wxString& aText,
SEVERITY aSeverity = RPT_SEVERITY_UNDEFINED ) override;
REPORTER& ReportTail( const wxString& aText, SEVERITY aSeverity = RPT_SEVERITY_UNDEFINED ) override;
REPORTER& ReportHead( const wxString& aText, SEVERITY aSeverity = RPT_SEVERITY_UNDEFINED ) override;
bool HasMessage() const override;
bool HasMessageOfSeverity( int aSeverityMask ) const override;
@ -66,9 +61,8 @@ private:
class KICOMMON_API WX_HTML_REPORT_PANEL : public WX_HTML_REPORT_PANEL_BASE
{
public:
WX_HTML_REPORT_PANEL( wxWindow* parent, wxWindowID id = wxID_ANY,
const wxPoint& pos = wxDefaultPosition,
const wxSize& size = wxSize( 500,300 ), long style = wxTAB_TRAVERSAL );
WX_HTML_REPORT_PANEL( wxWindow* parent, wxWindowID id = wxID_ANY, const wxPoint& pos = wxDefaultPosition,
const wxSize& size = wxSize( 500, 300 ), long style = wxTAB_TRAVERSAL );
~WX_HTML_REPORT_PANEL();
@ -85,8 +79,7 @@ public:
* @param aSeverity string classification level bitfield.
* @param aLocation REPORTER::LOCATION enum for placement of message.
*/
void Report( const wxString& aText, SEVERITY aSeverity,
REPORTER::LOCATION aLocation = REPORTER::LOC_BODY );
void Report( const wxString& aText, SEVERITY aSeverity, REPORTER::LOCATION aLocation = REPORTER::LOC_BODY );
/// Clears the report panel.
void Clear();
@ -100,30 +93,21 @@ public:
/// Set the lazy update. If this mode is on, messages are stored but the display
/// is not updated (Updating display can be very time consuming if there are many messages)
/// A call to Flush() will be needed after build the report
void SetLazyUpdate( bool aLazyUpdate );
void SetLazyUpdate( bool aLazyUpdate ) { m_lazyUpdate = aLazyUpdate; }
/// Force updating the HTML page, after the report is built in lazy mode
/// If aSort = true, the body messages will be ordered by severity
void Flush( bool aSort = false );
/// Set the visible severity filter.
/// if aSeverities < 0 the m_showAll option is set
void SetVisibleSeverities( int aSeverities );
/// @return the visible severity filter.
/// If the m_showAll option is set, the mask is < 0
int GetVisibleSeverities() const;
/// @return the visible severity filter.
/// If the m_showAll option is set, the mask is < 0
void SetShowSeverity( SEVERITY aSeverity, bool aValue );
/// Set the report full file name to the string.
void SetFileName( const wxString& aReportFileName );
void SetFileName( const wxString& aReportFileName ) { m_reportFileName = aReportFileName; }
/// @return reference to the current report fill file name string.
wxString& GetFileName( void );
wxString& GetFileName( void ) { return m_reportFileName; }
private:
struct REPORT_LINE
@ -134,21 +118,15 @@ private:
typedef std::vector<REPORT_LINE> REPORT_LINES;
wxString addHeader( const wxString& aBody );
wxString generateHtml( const REPORT_LINE& aLine );
wxString generatePlainText( const REPORT_LINE& aLine );
void updateBadges();
void scrollToBottom();
void syncCheckboxes();
void onRightClick( wxMouseEvent& event ) override;
void onMenuEvent( wxMenuEvent& event );
void onCheckBoxShowAll( wxCommandEvent& event ) override;
void onCheckBoxShowWarnings( wxCommandEvent& event ) override;
void onCheckBoxShowErrors( wxCommandEvent& event ) override;
void onCheckBoxShowInfos( wxCommandEvent& event ) override;
void onCheckBoxShowActions( wxCommandEvent& event ) override;
void onCheckBox( wxCommandEvent& event ) override;
void onBtnSaveToFile( wxCommandEvent& event ) override;
@ -161,10 +139,7 @@ private:
REPORT_LINES m_reportTail; ///< Lines to print at the end, regardless of sorting
REPORT_LINES m_reportHead; ///< ... and at the beginning, regardless of sorting
int m_severities; ///< message severities to display (mask)
bool m_lazyUpdate;
wxString m_reportFileName; ///< defaults to the not very useful /bin/report.txt
};
#endif //__WX_HTML_REPORT_PANEL_H__

20
common/widgets/wx_html_report_panel_base.cpp
View File

@ -88,11 +88,11 @@ WX_HTML_REPORT_PANEL_BASE::WX_HTML_REPORT_PANEL_BASE( wxWindow* parent, wxWindow
// Connect Events
m_htmlView->Connect( wxEVT_RIGHT_UP, wxMouseEventHandler( WX_HTML_REPORT_PANEL_BASE::onRightClick ), NULL, this );
m_checkBoxShowAll->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowAll ), NULL, this );
m_checkBoxShowErrors->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowErrors ), NULL, this );
m_checkBoxShowWarnings->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowWarnings ), NULL, this );
m_checkBoxShowActions->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowActions ), NULL, this );
m_checkBoxShowInfos->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowInfos ), NULL, this );
m_checkBoxShowAll->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowErrors->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowWarnings->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowActions->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowInfos->Connect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_btnSaveReportToFile->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onBtnSaveToFile ), NULL, this );
}
@ -100,11 +100,11 @@ WX_HTML_REPORT_PANEL_BASE::~WX_HTML_REPORT_PANEL_BASE()
{
// Disconnect Events
m_htmlView->Disconnect( wxEVT_RIGHT_UP, wxMouseEventHandler( WX_HTML_REPORT_PANEL_BASE::onRightClick ), NULL, this );
m_checkBoxShowAll->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowAll ), NULL, this );
m_checkBoxShowErrors->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowErrors ), NULL, this );
m_checkBoxShowWarnings->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowWarnings ), NULL, this );
m_checkBoxShowActions->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowActions ), NULL, this );
m_checkBoxShowInfos->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBoxShowInfos ), NULL, this );
m_checkBoxShowAll->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowErrors->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowWarnings->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowActions->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_checkBoxShowInfos->Disconnect( wxEVT_COMMAND_CHECKBOX_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onCheckBox ), NULL, this );
m_btnSaveReportToFile->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( WX_HTML_REPORT_PANEL_BASE::onBtnSaveToFile ), NULL, this );
}

10
common/widgets/wx_html_report_panel_base.fbp
View File

@ -273,7 +273,7 @@
<property name="window_extra_style"></property>
<property name="window_name"></property>
<property name="window_style"></property>
<event name="OnCheckBox">onCheckBoxShowAll</event>
<event name="OnCheckBox">onCheckBox</event>
</object>
</object>
<object class="sizeritem" expanded="true">
@ -349,7 +349,7 @@
<property name="window_extra_style"></property>
<property name="window_name"></property>
<property name="window_style"></property>
<event name="OnCheckBox">onCheckBoxShowErrors</event>
<event name="OnCheckBox">onCheckBox</event>
</object>
</object>
<object class="sizeritem" expanded="true">
@ -488,7 +488,7 @@
<property name="window_extra_style"></property>
<property name="window_name"></property>
<property name="window_style"></property>
<event name="OnCheckBox">onCheckBoxShowWarnings</event>
<event name="OnCheckBox">onCheckBox</event>
</object>
</object>
<object class="sizeritem" expanded="true">
@ -627,7 +627,7 @@
<property name="window_extra_style"></property>
<property name="window_name"></property>
<property name="window_style"></property>
<event name="OnCheckBox">onCheckBoxShowActions</event>
<event name="OnCheckBox">onCheckBox</event>
</object>
</object>
<object class="sizeritem" expanded="true">
@ -703,7 +703,7 @@
<property name="window_extra_style"></property>
<property name="window_name"></property>
<property name="window_style"></property>
<event name="OnCheckBox">onCheckBoxShowInfos</event>
<event name="OnCheckBox">onCheckBox</event>
</object>
</object>
<object class="sizeritem" expanded="true">

6
common/widgets/wx_html_report_panel_base.h
View File

@ -55,11 +55,7 @@ class KICOMMON_API WX_HTML_REPORT_PANEL_BASE : public wxPanel
// Virtual event handlers, override them in your derived class
virtual void onRightClick( wxMouseEvent& event ) { event.Skip(); }
virtual void onCheckBoxShowAll( wxCommandEvent& event ) { event.Skip(); }
virtual void onCheckBoxShowErrors( wxCommandEvent& event ) { event.Skip(); }
virtual void onCheckBoxShowWarnings( wxCommandEvent& event ) { event.Skip(); }
virtual void onCheckBoxShowActions( wxCommandEvent& event ) { event.Skip(); }
virtual void onCheckBoxShowInfos( wxCommandEvent& event ) { event.Skip(); }
virtual void onCheckBox( wxCommandEvent& event ) { event.Skip(); }
virtual void onBtnSaveToFile( wxCommandEvent& event ) { event.Skip(); }

2
eeschema/dialogs/dialog_annotate.cpp
View File

@ -193,8 +193,6 @@ void DIALOG_ANNOTATE::InitValues()
annotate_down_right_bitmap->SetBitmap( KiBitmapBundle( BITMAPS::annotate_down_right ) );
annotate_right_down_bitmap->SetBitmap( KiBitmapBundle( BITMAPS::annotate_right_down ) );
m_MessageWindow->SetVisibleSeverities( cfg->m_AnnotatePanel.messages_filter );
m_MessageWindow->MsgPanelSetMinSize( wxSize( -1, 160 ) );
}

3
pcbnew/dialogs/dialog_board_reannotate.cpp
View File

@ -118,9 +118,6 @@ DIALOG_BOARD_REANNOTATE::DIALOG_BOARD_REANNOTATE( PCB_EDIT_FRAME* aParentFrame )
m_GridChoice->Set( gridslist );
for( wxRadioButton* button : m_scopeRadioButtons )
button->SetValue( false );
m_ExcludeList->SetToolTip( m_ExcludeListText->GetToolTipText() );
m_GridChoice->SetToolTip( m_SortGridText->GetToolTipText() );

2
pcbnew/dialogs/dialog_board_reannotate_base.cpp
View File

@ -294,7 +294,7 @@ DIALOG_BOARD_REANNOTATE_BASE::DIALOG_BOARD_REANNOTATE_BASE( wxWindow* parent, wx
m_MessageWindow = new WX_HTML_REPORT_PANEL( this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxTAB_TRAVERSAL );
m_MessageWindow->SetMinSize( wxSize( -1,300 ) );
bSizerMessages->Add( m_MessageWindow, 1, wxEXPAND|wxLEFT|wxRIGHT, 5 );
bSizerMessages->Add( m_MessageWindow, 1, wxEXPAND|wxLEFT|wxRIGHT, 10 );
bmainSizer->Add( bSizerMessages, 1, wxEXPAND|wxTOP, 15 );

2
pcbnew/dialogs/dialog_board_reannotate_base.fbp
View File

@ -2969,7 +2969,7 @@
<property name="orient">wxVERTICAL</property>
<property name="permission">none</property>
<object class="sizeritem" expanded="false">
<property name="border">5</property>
<property name="border">10</property>
<property name="flag">wxEXPAND|wxLEFT|wxRIGHT</property>
<property name="proportion">1</property>
<object class="wxPanel" expanded="false">

14
pcbnew/dialogs/dialog_drc.cpp
View File

@ -1271,8 +1271,6 @@ void DIALOG_DRC::updateDisplayedCounts()
numExcluded += m_fpWarningsProvider->GetCount( RPT_SEVERITY_EXCLUSION );
}
bool showErrors = m_showErrors->GetValue();
bool showWarnings = m_showWarnings->GetValue();
bool errorsOverflowed = false;
bool warningsOverflowed = false;
bool markersOverflowed = false;
@ -1292,9 +1290,9 @@ void DIALOG_DRC::updateDisplayedCounts()
if( ii == DRCE_UNCONNECTED_ITEMS )
{
if( showWarnings && severity == RPT_SEVERITY_WARNING )
if( m_showWarnings->GetValue() && severity == RPT_SEVERITY_WARNING )
unconnectedOverflowed = true;
else if( showErrors && severity == RPT_SEVERITY_ERROR )
else if( m_showErrors->GetValue() && severity == RPT_SEVERITY_ERROR )
unconnectedOverflowed = true;
}
else if( ii == DRCE_MISSING_FOOTPRINT
@ -1304,16 +1302,16 @@ void DIALOG_DRC::updateDisplayedCounts()
|| ii == DRCE_SCHEMATIC_PARITY
|| ii == DRCE_FOOTPRINT_FILTERS )
{
if( showWarnings && severity == RPT_SEVERITY_WARNING )
if( m_showWarnings->GetValue() && severity == RPT_SEVERITY_WARNING )
footprintsOverflowed = true;
else if( showErrors && severity == RPT_SEVERITY_ERROR )
else if( m_showErrors->GetValue() && severity == RPT_SEVERITY_ERROR )
footprintsOverflowed = true;
}
else
{
if( showWarnings && severity == RPT_SEVERITY_WARNING )
if( m_showWarnings->GetValue() && severity == RPT_SEVERITY_WARNING )
markersOverflowed = true;
else if( showErrors && severity == RPT_SEVERITY_ERROR )
else if( m_showErrors->GetValue() && severity == RPT_SEVERITY_ERROR )
markersOverflowed = true;
}
}