mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

clang-tidy readability-simplify-boolean-expr

Browse Source 
Change-Id: Iea7ab64683f0b29794d50d774cc482b54a00e70a
Reviewed-on: https://gerrit.libreoffice.org/36450
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Noel Grandin
2017-04-12 11:21:58 +02:00
parent 5bcd18461b
commit 9348b322a5
38 changed files with 74 additions and 242 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
scripting/source/vbaevents/eventhelper.cxx
View File

@@ -87,18 +87,12 @@ static const sal_Int32 DELIMLEN = strlen(DELIM);
bool isKeyEventOk( awt::KeyEvent& evt, const Sequence< Any >& params )
{
if ( !( params.getLength() > 0 ) ||
!( params[ 0 ] >>= evt ) )
return false;
return true;
return ( params.getLength() > 0 ) && ( params[ 0 ] >>= evt );
}
bool isMouseEventOk( awt::MouseEvent& evt, const Sequence< Any >& params )
{
if ( !( params.getLength() > 0 ) ||
!( params[ 0 ] >>= evt ) )
return false;
return true;
return ( params.getLength() > 0 ) && ( params[ 0 ] >>= evt );
}
Sequence< Any > ooMouseEvtToVBADblClick( const Sequence< Any >& params )
@@ -793,14 +787,7 @@ bool DenyMouseDrag(const ScriptEvent& evt, void const * )
{
awt::MouseEvent aEvent;
evt.Arguments[ 0 ] >>= aEvent;
if (aEvent.Buttons == 0 )
{
return true;
}
else
{
return false;
}
return aEvent.Buttons == 0;
}