diff options
author | Chris Wilson <chris+github@qwirx.com> | 2007-03-05 00:06:26 +0000 |
---|---|---|
committer | Chris Wilson <chris+github@qwirx.com> | 2007-03-05 00:06:26 +0000 |
commit | d17445865a24087fc7df683a957783c529a232c1 (patch) | |
tree | 93d3e9418427ed0d4eb5a582990e9933a13bf89c /lib | |
parent | 3bd5d45687714bd7402c26f1aea915bcc15f0481 (diff) |
Add logging in remaining places where Common OSFileOpenError can be thrown.
Fix compile warning (signed vs unsigned comparison)
(refs #3, merges part of [1099], and [1368])
Diffstat (limited to 'lib')
-rw-r--r-- | lib/common/FileStream.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/lib/common/FileStream.cpp b/lib/common/FileStream.cpp index 8040cab4..e0806e10 100644 --- a/lib/common/FileStream.cpp +++ b/lib/common/FileStream.cpp @@ -10,6 +10,7 @@ #include "Box.h" #include "FileStream.h" #include "CommonException.h" +#include "Logging.h" #include <errno.h> @@ -73,6 +74,7 @@ FileStream::FileStream(tOSFileHandle FileDescriptor) #endif { MEMLEAKFINDER_NOT_A_LEAK(this); + BOX_ERROR("FileStream: called with invalid file handle"); THROW_EXCEPTION(CommonException, OSFileOpenError) } #ifdef WIN32 @@ -100,6 +102,7 @@ FileStream::FileStream(const FileStream &rToCopy) #endif { MEMLEAKFINDER_NOT_A_LEAK(this); + BOX_ERROR("FileStream: copying unopened file"); THROW_EXCEPTION(CommonException, OSFileOpenError) } } @@ -157,8 +160,8 @@ int FileStream::Read(void *pBuffer, int NBytes, int Timeout) } else { - ::syslog(LOG_ERR, "Failed to read from file: error %d", - GetLastError()); + BOX_ERROR("Failed to read from file: " << + GetErrorMessage(GetLastError())); r = -1; } #else @@ -222,7 +225,7 @@ void FileStream::Write(const void *pBuffer, int NBytes) NULL ); - if ( (res == 0) || (numBytesWritten != NBytes)) + if ((res == 0) || (numBytesWritten != (DWORD)NBytes)) { // DWORD err = GetLastError(); THROW_EXCEPTION(CommonException, OSFileWriteError) |