diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2017-06-19 12:24:26 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2017-06-19 12:24:26 +0200 |
| commit | bb053e86738d69af603b222532a898d355f51eec (patch) | |
| tree | e187f02e201530d5ade61d7021c45aeb8767c4b7 /sal | |
| parent | 45c06838e95c94445359536d84c6328fa8b17a66 (diff) | |
Remove questionable asserts
Both osl_add/removeSignalHandler apparently handle !handler gracefully (the
former even including explicit code for that case, which doesn't fit with having
an assert), and before bca760c294c256fcb1fd3649dbf50f61a4bda474 "tdf#43157 -
osl: convert OSL_ASSERTs to assert in signalshared.cxx" those asserts were
merely OSL_ASSERTs. So keep this published API accepting !handler (even if the
documentation is silent on this).
Change-Id: I63512916a66a2a58b932174ec5d92ce612c236a2
Diffstat (limited to 'sal')
| -rw-r--r-- | sal/osl/all/signalshared.cxx | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/sal/osl/all/signalshared.cxx b/sal/osl/all/signalshared.cxx index 816f20a2c808..9cf7fafc2de5 100644 --- a/sal/osl/all/signalshared.cxx +++ b/sal/osl/all/signalshared.cxx @@ -17,7 +17,6 @@ * the License at http://www.apache.org/licenses/LICENSE-2.0 . */ -#include <cassert> #include <sal/config.h> #include <signalshared.hxx> @@ -68,8 +67,6 @@ oslSignalAction callSignalHandler(oslSignalInfo* pInfo) oslSignalHandler SAL_CALL osl_addSignalHandler(oslSignalHandlerFunction handler, void* pData) { - assert(handler); - if (!handler) return nullptr; @@ -98,8 +95,6 @@ oslSignalHandler SAL_CALL osl_addSignalHandler(oslSignalHandlerFunction handler, sal_Bool SAL_CALL osl_removeSignalHandler(oslSignalHandler handler) { - assert(handler); - if (!bInitSignal) bInitSignal = initSignal(); |