From 532a29d12d72f54549d8b71edd485c17af65698a Mon Sep 17 00:00:00 2001 From: Francis Murtagh Date: Mon, 29 Jun 2020 11:50:01 +0100 Subject: IVGCVSW-4487 Remove boost::filesystem * Replace filesystem::path * Replace filesystem::exists * Replace filesystem::is_directory * Replace filesystem::directory_iterator * Replace filesystem::filesystem_error exception * Replace filesystem::temp_directory_path * Replace filesystem::unique path * Replace filesystem::ofstream with std::ofstream * Replace filesystem::remove * Replace filesystem::is_regular_file * Replace boost::optional with armnn::Optional in touched files * Remove some superfluous includes * Update build guides, GlobalConfig.cmake and CMakeLists.txt * Remove redundant armnnUtils::Filesystem::Remove function. * Remove redundant armnnUtils::Filesystem::GetFileSize function. Temporarily adding back Boost::filesystem to enable Boost::dll. Signed-off-by: Francis Murtagh Signed-off-by: Colm Donelan Change-Id: Ifa46d4a0097d2612ddacd8e9736c0b36e365fb11 --- src/armnnUtils/Filesystem.cpp | 53 ++++++++++----------------------- src/armnnUtils/Filesystem.hpp | 5 ++-- src/armnnUtils/ModelAccuracyChecker.cpp | 1 - 3 files changed, 18 insertions(+), 41 deletions(-) (limited to 'src/armnnUtils') diff --git a/src/armnnUtils/Filesystem.cpp b/src/armnnUtils/Filesystem.cpp index 4a2e2ed393..ac9a414ae4 100644 --- a/src/armnnUtils/Filesystem.cpp +++ b/src/armnnUtils/Filesystem.cpp @@ -5,51 +5,30 @@ #include "Filesystem.hpp" -#if defined(__unix__) -#include -#include -#elif defined(_MSC_VER) -#include "WindowsWrapper.hpp" -#endif - namespace armnnUtils { namespace Filesystem { -long long GetFileSize(const char* path) +/** + * @brief Construct a temporary file name. + * + * Given a specified file name construct a path to that file in the + * system temporary directory. If the file already exists it is deleted. This + * could throw filesystem_error exceptions. + * + * @param fileName the file name required in the temporary directory. + * @return path consisting of system temporary directory and file name. + */ +fs::path NamedTempFile(const char* fileName) { -#if defined(__ANDROID__) - struct stat statusBuffer; - if (stat(path, & statusBuffer) != 0) - { - return -1; - } - return statusBuffer.st_size; -#elif defined(__unix__) - struct stat statusBuffer; - if (stat(path, & statusBuffer) != 0) - { - return -1; - } - return static_cast(statusBuffer.st_size); -#elif defined(_MSC_VER) - WIN32_FILE_ATTRIBUTE_DATA attr; - if (::GetFileAttributesEx(path, GetFileExInfoStandard, &attr) == 0) + fs::path tmpDir = fs::temp_directory_path(); + fs::path namedTempFile{tmpDir / fileName}; + if (fs::exists(namedTempFile)) { - return -1; + fs::remove(namedTempFile); } - return attr.nFileSizeLow; -#endif -} - -bool Remove(const char* path) -{ -#if defined(__unix__) - return remove(path) == 0; -#elif defined(_MSC_VER) - return ::DeleteFile(path); -#endif + return namedTempFile; } } diff --git a/src/armnnUtils/Filesystem.hpp b/src/armnnUtils/Filesystem.hpp index 1b12502b40..869b0c1b5e 100644 --- a/src/armnnUtils/Filesystem.hpp +++ b/src/armnnUtils/Filesystem.hpp @@ -13,9 +13,8 @@ namespace armnnUtils namespace Filesystem { -long long GetFileSize(const char* path); - -bool Remove(const char* path); +/// Returns a path to a file in the system temporary folder. If the file existed it will be deleted. +fs::path NamedTempFile(const char* fileName); } } diff --git a/src/armnnUtils/ModelAccuracyChecker.cpp b/src/armnnUtils/ModelAccuracyChecker.cpp index d197dc86ba..418737c0f6 100644 --- a/src/armnnUtils/ModelAccuracyChecker.cpp +++ b/src/armnnUtils/ModelAccuracyChecker.cpp @@ -7,7 +7,6 @@ #include -#include #include #include -- cgit v1.2.1