Fixed an oversight of yesterday's restructuring:

* FDCloser is private, so it should live in BPackageKit::BHPKG::BPrivate
* adjusted 'package' binary to no longer use FDCloser (trivial)
* minor cleanup in command_extract.cpp

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@40329 a95241bf-73f2-0310-859d-f6bbb57e9c96

headers/private/package/hpkg/FDCloser.h

@@ -2,14 +2,16 @@
  * Copyright 2009, Ingo Weinhold, [email protected].
  * Distributed under the terms of the MIT License.
  */
-#ifndef _PACKAGE__HPKG__FD_CLOSER_H_
-#define _PACKAGE__HPKG__FD_CLOSER_H_
+#ifndef _PACKAGE__HPKG__PRIVATE__FD_CLOSER_H_
+#define _PACKAGE__HPKG__PRIVATE__FD_CLOSER_H_
 
 
 namespace BPackageKit {
 
 namespace BHPKG {
 
+namespace BPrivate {
+
 
 struct FDCloser {
 	FDCloser(int fd)
@@ -29,9 +31,11 @@ private:
 };
 
 
+}	// namespace BPrivate
+
 }	// namespace BHPKG
 
 }	// namespace BPackageKit
 
 
-#endif	// _PACKAGE__HPKG__FD_CLOSER_H_
+#endif	// _PACKAGE__HPKG__PRIVATE__FD_CLOSER_H_

src/bin/package/command_extract.cpp

@@ -21,7 +21,6 @@
 
 #include <AutoDeleter.h>
 
-#include <package/hpkg/FDCloser.h>
 #include <package/hpkg/PackageContentHandler.h>
 #include <package/hpkg/PackageDataReader.h>
 #include <package/hpkg/PackageEntry.h>
@@ -198,7 +197,6 @@ struct PackageContentExtractHandler : BPackageContentHandler {
 					strerror(errno));
 				return errno;
 		}
-		FDCloser fdCloser(fd);
 
 		// write data
 		status_t error;
@@ -210,10 +208,9 @@ struct PackageContentExtractHandler : BPackageContentHandler {
 		} else
 			error = _ExtractFileData(&fPackageFileReader, data, fd);
 
-		if (error != B_OK)
-			return error;
+		close(fd);
 
-		return B_OK;
+		return error;
 	}
 
 	virtual status_t HandleEntryDone(BPackageEntry* entry)