libutil: make AutoCloseFD a better resource

add a reset() method to close the wrapped fd instead of assigning magic
constants. also make the from-fd constructor explicit so you can't
accidentally assign the *wrong* magic constant, or even an unrelated
integer that also just happens to be an fd by pure chance.

Change-Id: I51311b0f6e040240886b5103d39d1794a6acc325

1 files changed, 1 insertions, 1 deletions

diff --git a/src/resolve-system-dependencies/resolve-system-dependencies.cc b/src/resolve-system-dependencies/resolve-system-dependencies.cc
index 424c789d8..2c4b06791 100644
--- a/src/resolve-system-dependencies/resolve-system-dependencies.cc
+++ b/src/resolve-system-dependencies/resolve-system-dependencies.cc
@@ -30,7 +30,7 @@ std::set<std::string> readCacheFile(const Path & file)
 
 std::set<std::string> runResolver(const Path & filename)
 {
-    AutoCloseFD fd = open(filename.c_str(), O_RDONLY);
+    AutoCloseFD fd{open(filename.c_str(), O_RDONLY)};
     if (!fd)
         throw SysError("opening '%s'", filename);