Standardize view checking

Uses wxCHECK* functions for mesage display and correct behavior when
encountering an invalid layer
This commit is contained in:
Seth Hillbrand 2023-01-27 12:35:36 -08:00
parent ff46b7b1e5
commit 9ad2e9f25d
1 changed files with 4 additions and 2 deletions

View File

@ -333,8 +333,8 @@ void VIEW::Add( VIEW_ITEM* aItem, int aDrawPriority )
for( int i = 0; i < layers_count; ++i )
{
if( layers[i] < 0 || layers[i] >= (int)m_layers.size() )
continue;
wxCHECK2_MSG( layers[i] >= 0 && static_cast<unsigned>( layers[i] ) < m_layers.size(),
continue, wxS( "Invalid layer" ) );
VIEW_LAYER& l = m_layers[layers[i]];
l.items->Insert( aItem );
@ -1451,6 +1451,8 @@ void VIEW::UpdateItems()
for( int i = 0; i < layers_count; ++i )
{
wxCHECK2_MSG( layers[i] >= 0 && static_cast<unsigned>( layers[i] ) < m_layers.size(),
continue, wxS( "Invalid layer" ) );
VIEW_LAYER& l = m_layers[layers[i]];
l.items->Insert( item );
MarkTargetDirty( l.target );