remove some range checks
which are more likely to hide an underlying bug than they are to help Change-Id: I5087827665bd6213142ee90450bf58d8d819c93f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/178005 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
parent
3339fd9a3f
commit
b0d6329c05
1 changed files with 1 additions and 4 deletions
|
@ -138,10 +138,7 @@ void MenuItemList::InsertSeparator(const OUString &rIdent, size_t nPos)
|
||||||
|
|
||||||
void MenuItemList::Remove( size_t nPos )
|
void MenuItemList::Remove( size_t nPos )
|
||||||
{
|
{
|
||||||
if( nPos < maItemList.size() )
|
maItemList.erase( maItemList.begin() + nPos );
|
||||||
{
|
|
||||||
maItemList.erase( maItemList.begin() + nPos );
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
void MenuItemList::Clear()
|
void MenuItemList::Clear()
|
||||||
|
|
Loading…
Reference in a new issue