diff options
author | Daniel Bankston <daniel.dev.libreoffice@gmail.com> | 2012-04-02 19:35:34 -0500 |
---|---|---|
committer | Luboš Luňák <l.lunak@suse.cz> | 2012-04-03 13:41:40 +0200 |
commit | 2d691f26c6d4fd6fac11559f9b5b854d306e137a (patch) | |
tree | 79328fe375541f390e4f81ca5e35fd6444aab794 /vcl | |
parent | fdo#44610 - EasyHack (diff) | |
download | core-2d691f26c6d4fd6fac11559f9b5b854d306e137a.tar.gz core-2d691f26c6d4fd6fac11559f9b5b854d306e137a.zip |
Ignore return type of Region::Intersect
Caller of Region::Intersect was not ignore the return value in a conditional, so I refactored the conditional to not depend on the return value of Intersect without affect the original functionality of the code. This usage of Intersect may be a possible defect and should be checked out.
Diffstat (limited to 'vcl')
-rw-r--r-- | vcl/source/control/edit.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/vcl/source/control/edit.cxx b/vcl/source/control/edit.cxx index 7d033164bd63..b3f75167fc80 100644 --- a/vcl/source/control/edit.cxx +++ b/vcl/source/control/edit.cxx @@ -670,7 +670,8 @@ void Edit::ImplRepaint( xub_StrLen nStart, xub_StrLen nEnd, bool bLayout ) nIndex++; } i = nIndex; - if( aClip.Intersect( aRegion ) && nAttr ) + aClip.Intersect(aRegion); + if( nAttr ) { Font aFont = GetFont(); if ( nAttr & EXTTEXTINPUT_ATTR_UNDERLINE ) |