These arguments can apparently never be null

Fixing 7b4f4f15 "osl: followup to 7c6ccc42 for
w32/unx file.cxx" for good...

Change-Id: Icedaaa0b0f909d802dbdcf4fdaa40fd338bcbf11
Reviewed-on: https://gerrit.libreoffice.org/39892Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

sal/osl/unx/file.cxx

@@ -200,12 +200,10 @@ FileHandle_Impl::Allocator::~Allocator()
 
 void FileHandle_Impl::Allocator::allocate(sal_uInt8 **ppBuffer, size_t *pnSize)
 {
-    SAL_WARN_IF((!ppBuffer) || (!pnSize), "sal.osl", "FileHandle_Impl::Allocator::allocate(): contract violation");
-    if (ppBuffer && pnSize)
-    {
-        *ppBuffer = static_cast< sal_uInt8* >(rtl_cache_alloc(m_cache));
-        *pnSize = m_bufsiz;
-    }
+    assert(ppBuffer != nullptr);
+    assert(pnSize != nullptr);
+    *ppBuffer = static_cast< sal_uInt8* >(rtl_cache_alloc(m_cache));
+    *pnSize = m_bufsiz;
 }
 
 void FileHandle_Impl::Allocator::deallocate(sal_uInt8 * pBuffer)

sal/osl/w32/file.cxx

@@ -31,6 +31,7 @@
 #include "file_url.hxx"
 #include "file_error.hxx"
 
+#include <cassert>
 #include <cstdio>
 #include <algorithm>
 #include <limits>
@@ -180,9 +181,9 @@ FileHandle_Impl::Allocator::~Allocator()
 
 void FileHandle_Impl::Allocator::allocate (sal_uInt8 **ppBuffer, SIZE_T * pnSize)
 {
-    SAL_WARN_IF((!ppBuffer) || (!pnSize), "sal.osl", "FileHandle_Impl::Allocator::allocate(): contract violation");
+    assert(ppBuffer != nullptr);
+    assert(pnSize != nullptr);
     *ppBuffer = static_cast< sal_uInt8* >(rtl_cache_alloc(m_cache));
-
     *pnSize = m_bufsiz;
 }
 
@@ -195,13 +196,12 @@ void FileHandle_Impl::Allocator::deallocate (sal_uInt8 * pBuffer)
 FileHandle_Impl::Guard::Guard(LPCRITICAL_SECTION pMutex)
     : m_mutex (pMutex)
 {
-    SAL_WARN_IF(!(m_mutex), "sal.osl", "FileHandle_Impl::Guard::Guard(): null pointer.");
+    assert(pMutex != nullptr);
     ::EnterCriticalSection (m_mutex);
 }
 
 FileHandle_Impl::Guard::~Guard()
 {
-    SAL_WARN_IF(!(m_mutex), "sal.osl", "FileHandle_Impl::Guard::~Guard(): null pointer.");
     ::LeaveCriticalSection (m_mutex);
 }