File.cpp

Index: File.cpp
===================================================================
RCS file: /cvsroot/linuxdcpp/linuxdcpp/client/File.cpp,v
retrieving revision 1.4
diff -u -r1.4 File.cpp
--- File.cpp	7 Jul 2007 21:21:06 -0000	1.4
+++ File.cpp	7 Oct 2007 19:17:59 -0000
@@ -223,8 +223,9 @@
 	}
 
 	h = open(filename.c_str(), m, S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH);
-	if(h == -1)
-		throw FileException("Could not open file");
+	if(h == -1) {
+		throw FileException("Could not open file: "+Util::translateError(errno));
+	}
 }
 
 uint32_t File::getLastModified() throw() {
@@ -271,20 +272,22 @@
 }
 
 size_t File::read(void* buf, size_t& len) throw(FileException) {
-	ssize_t x = ::read(h, buf, len);
-	if(x == -1)
-		throw FileException("Read error");
-	len = x;
-	return (size_t)x;
+	ssize_t result = ::read(h, buf, len);
+	if (result == -1) {
+		throw FileException("Read error: "+Util::translateError(errno));
+	}
+	len = result;
+	return (size_t)result;
 }
 
 size_t File::write(const void* buf, size_t len) throw(FileException) {
-	ssize_t x = ::write(h, buf, len);
-	if(x == -1)
-		throw FileException("Write error");
-	if(x < (ssize_t)len)
-		throw FileException("Disk full(?)");
-	return x;
+	ssize_t result = ::write(h, buf, len);
+
+	if (result == -1) {
+		throw FileException("Write error: "+Util::translateError(errno));
+	}
+	
+	return result;
 }
 
 // some ftruncate implementations can't extend files like SetEndOfFile,
@@ -338,10 +341,10 @@
 void File::renameFile(const string& source, const string& target) throw(FileException) {
 	int ret = ::rename(Text::fromUtf8(source).c_str(), Text::fromUtf8(target).c_str());
 	if(ret != 0 && errno == EXDEV) {
-		copyFile(source.c_str(), target.c_str());
-		deleteFile(source.c_str());
+		copyFile(source, target);
+		deleteFile(source);
 	} else if(ret != 0)
-		throw FileException(source.c_str() + Util::translateError(errno));
+		throw FileException(source + Util::translateError(errno));
 }
 
 // This doesn't assume all bytes are written in one write call, it is a bit safer