diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-01-26 15:41:06 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-01-26 15:59:29 +0000 |
commit | 237fe9d1d62d215721f120f5ec6399fc1456119a (patch) | |
tree | 64fd0aacaeb15a97b917490277c6e9e5e8bd79cf /searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp | |
parent | 972de79bfeebf0d9f78a28067969494536d65ca5 (diff) |
- assert result of Close() in destructor of FastOS_File and FastOS_BufferedFile.
- Check result of Close()
- Check result of Sync()
- Scope FastOS_File to avoid explicit Close().
- NULL -> nullptr
Diffstat (limited to 'searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp')
-rw-r--r-- | searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp b/searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp index 15c7e114761..7ea480bf542 100644 --- a/searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp +++ b/searchlib/src/tests/vespa-fileheader-inspect/vespa-fileheader-inspect_test.cpp @@ -17,7 +17,6 @@ bool writeHeader(const FileHeader &header, const vespalib::string &fileName) { if (!EXPECT_EQUAL(header.getSize(), header.writeFile(file))) { return false; } - file.Close(); return true; } @@ -30,7 +29,6 @@ vespalib::string readFile(const vespalib::string &fileName) { EXPECT_LESS(len, sizeof(buf)); // make sure we got everything vespalib::string str(buf, len); - file.Close(); return str; } |