diff options
| author | Douglas Mencken <dougmencken@gmail.com> | 2014-12-01 12:33:56 -0500 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-12-02 20:26:01 +0000 |
| commit | 0d0729ca97e67505a52dcb07d7819f6c444a0031 (patch) | |
| tree | 27a1967f91ffa45ea2d55c61c618645352030658 /vcl/osx/salframeview.mm | |
| parent | 13658762d2a6a0f33ad23ab6b9295ee93d2280bc (diff) | |
prefer `if (s)' to `if (nil != s)' for objective-c
in objc, explicit comparison with nil isn't necessary and can produce errors like:
comparison between distinct pointer types (...) and 'objc_object*' lacks a cast
upd: thanks Norbert Thiebaud for suggestion
``hitChild is a Reference so hitChild = nil was !hitChild.is() right?''
Change-Id: I105be50e5a37bb63e360622e590ec4916fa8a84f
Reviewed-on: https://gerrit.libreoffice.org/11891
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'vcl/osx/salframeview.mm')
| -rw-r--r-- | vcl/osx/salframeview.mm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vcl/osx/salframeview.mm b/vcl/osx/salframeview.mm index aca5e8dc4a4d..add9891b6f86 100644 --- a/vcl/osx/salframeview.mm +++ b/vcl/osx/salframeview.mm @@ -1731,7 +1731,7 @@ private: -(::com::sun::star::accessibility::XAccessibleContext *)accessibleContext { - if ( mpReferenceWrapper == nil ) { + if ( !mpReferenceWrapper ) { // some frames never become visible .. ::vcl::Window *pWindow = mpFrame -> GetWindow(); if ( ! pWindow ) |