commit 986fab1496defebdff7986939394d84f48c389e2
author andrew@webrtc.org <andrew@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Thu Dec 15 19:11:41 2011 +0000
committer andrew@webrtc.org <andrew@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d> Thu Dec 15 19:11:41 2011 +0000
tree 1cf5b74d42c0928394954878a471132d16e434a7
parent 0c839fe8738b906c3d8a88817532a64670e73e83

Clean up file wrapper a bit further.

- Make error handling in Read, Write and WriteText consistent.
- Improve the interface comments a bit.

TEST=voe_auto_test, vie_auto_test

Review URL: http://webrtc-codereview.appspot.com/321012

git-svn-id: http://webrtc.googlecode.com/svn/trunk@1210 4adac7df-926f-26a2-2b94-8c16560cd09d

src/system_wrappers/source/file_impl.cc

diff --git a/src/system_wrappers/source/file_impl.cc b/src/system_wrappers/source/file_impl.cc
index 67a4567..d163bf6 100644
--- a/src/system_wrappers/source/file_impl.cc
+++ b/src/system_wrappers/source/file_impl.cc
@@ -88,23 +88,24 @@
 int FileWrapperImpl::FileName(char* fileNameUTF8,
                               size_t size) const
 {
-    size_t len = strlen(_fileNameUTF8);
-    if(len > kMaxFileNameSize)
+    size_t length = strlen(_fileNameUTF8);
+    if(length > kMaxFileNameSize)
     {
         assert(false);
         return -1;
     }
-    if(len < 1)
+    if(length < 1)
     {
         return -1;
     }
+
     // Make sure to NULL terminate
-    if(size < len)
+    if(size < length)
     {
-        len = size - 1;
+        length = size - 1;
     }
-    memcpy(fileNameUTF8, _fileNameUTF8, len);
-    fileNameUTF8[len] = 0;
+    memcpy(fileNameUTF8, _fileNameUTF8, length);
+    fileNameUTF8[length] = 0;
     return 0;
 }
 
@@ -186,38 +187,33 @@
     return -1;
 }
 
-int FileWrapperImpl::Read(void *buf, int len)
+int FileWrapperImpl::Read(void* buf, int length)
 {
-    if (len < 0)
-    {
-        return -1;
-    }
-    if (_id != NULL)
-    {
-        int res = static_cast<int>(fread(buf, 1, len, _id));
-        if (res != len)
-        {
-            if(!_looping)
-            {
-                CloseFile();
-            }
-        }
-        return res;
-    }
-    return -1;
-}
-
-int FileWrapperImpl::WriteText(const char* format, ...)
-{
-    if (_readOnly)
+    if (length < 0)
         return -1;
 
     if (_id == NULL)
         return -1;
 
+    int bytes_read = static_cast<int>(fread(buf, 1, length, _id));
+    if (bytes_read != length && !_looping)
+    {
+        CloseFile();
+    }
+    return bytes_read;
+}
+
+int FileWrapperImpl::WriteText(const char* format, ...)
+{
     if (format == NULL)
         return -1;
 
+    if (_readOnly)
+        return -1;
+
+    if (_id == NULL)
+        return -1;
+
     va_list args;
     va_start(args, format);
     int num_chars = vfprintf(_id, format, args);
@@ -234,30 +230,35 @@
     }
 }
 
-bool FileWrapperImpl::Write(const void* buf, int len)
+bool FileWrapperImpl::Write(const void* buf, int length)
 {
+    if (buf == NULL)
+        return false;
+
+    if (length < 0)
+        return false;
+
     if (_readOnly)
+        return false;
+
+    if (_id == NULL)
+        return false;
+
+    // Check if it's time to stop writing.
+    if (_maxSizeInBytes > 0 && (_sizeInBytes + length) > _maxSizeInBytes)
     {
+        Flush();
         return false;
     }
 
-    if (_id != NULL)
+    size_t num_bytes = fwrite(buf, 1, length, _id);
+    if (num_bytes > 0)
     {
-        // Check if it's time to stop writing.
-        if (_maxSizeInBytes > 0 && (_sizeInBytes + len) > _maxSizeInBytes)
-        {
-            Flush();
-            return false;
-        }
-
-        size_t num_bytes = fwrite(buf, 1, len, _id);
-        if (num_bytes > 0)
-        {
-            _sizeInBytes += num_bytes;
-            return true;
-        }
-        CloseFile();
+        _sizeInBytes += num_bytes;
+        return true;
     }
+
+    CloseFile();
     return false;
 }