diff options
author | Noel Grandin <noel@peralex.com> | 2016-04-25 09:59:16 +0200 |
---|---|---|
committer | Ashod Nakashian <ashod.nakashian@collabora.co.uk> | 2016-07-12 22:06:19 -0400 |
commit | c5cdfe39d3afca716e7b11f8f8e169ce378861ea (patch) | |
tree | 5b804fa2e78405afe16db734f88f24a7b9714209 /basctl | |
parent | 0c4c31576fe03b3b59217bf9990b75f32d193d0d (diff) |
update loplugin stylepolice to check local pointers vars
are actually pointer vars.
Also convert from regex to normal code, so we can enable this
plugin all the time.
Reviewed-on: https://gerrit.libreoffice.org/24391
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
(cherry picked from commit e8fd5a07eca70912ddee45aaa34d434809b59fb7)
Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1
Diffstat (limited to 'basctl')
-rw-r--r-- | basctl/source/basicide/breakpoint.cxx | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/basctl/source/basicide/breakpoint.cxx b/basctl/source/basicide/breakpoint.cxx index 5c59bea38d5a..411b99475d6d 100644 --- a/basctl/source/basicide/breakpoint.cxx +++ b/basctl/source/basicide/breakpoint.cxx @@ -42,8 +42,8 @@ BreakPointList::~BreakPointList() void BreakPointList::reset() { - for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i ) - delete maBreakPoints[ i ]; + for (BreakPoint* pBreakPoint : maBreakPoints) + delete pBreakPoint; maBreakPoints.clear(); } @@ -74,9 +74,8 @@ void BreakPointList::SetBreakPointsInBasic(SbModule* pModule) { pModule->ClearAllBP(); - for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i ) + for (BreakPoint* pBrk : maBreakPoints) { - BreakPoint* pBrk = maBreakPoints[ i ]; if ( pBrk->bEnabled ) pModule->SetBP( (sal_uInt16)pBrk->nLine ); } @@ -84,9 +83,8 @@ void BreakPointList::SetBreakPointsInBasic(SbModule* pModule) BreakPoint* BreakPointList::FindBreakPoint(size_t nLine) { - for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i ) + for (BreakPoint* pBrk : maBreakPoints) { - BreakPoint* pBrk = maBreakPoints[ i ]; if ( pBrk->nLine == nLine ) return pBrk; } @@ -127,9 +125,8 @@ void BreakPointList::AdjustBreakPoints(size_t nLine, bool bInserted) void BreakPointList::ResetHitCount() { - for ( size_t i = 0, n = maBreakPoints.size(); i < n; ++i ) + for (BreakPoint* pBrk : maBreakPoints) { - BreakPoint* pBrk = maBreakPoints[ i ]; pBrk->nHitCount = 0; } } |