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 | 93acb28e10760513b8bffd2c36dcbd68e75e57c7 (diff) | |
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 ) |