tequilaOS/platform_system_core
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Don't use TEMP_FAILURE_RETRY on close()"

Browse source
This commit is contained in:
Nick Kralevich 2015-05-20 16:04:36 +00:00 committed by Gerrit Code Review
commit 6a8b7b65f5
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
base/file.cpp
View file

@ -51,7 +51,7 @@ bool ReadFileToString(const std::string& path, std::string* content) {
return false;
}
bool result = ReadFdToString(fd, content);
TEMP_FAILURE_RETRY(close(fd));
close(fd);
return result;
}
@ -102,7 +102,7 @@ bool WriteStringToFile(const std::string& content, const std::string& path,
ALOGE("android::WriteStringToFile write failed: %s", strerror(errno));
return CleanUpAfterFailedWrite(path);
}
TEMP_FAILURE_RETRY(close(fd));
close(fd);
return true;
}
#endif
@ -116,7 +116,7 @@ bool WriteStringToFile(const std::string& content, const std::string& path) {
}
bool result = WriteStringToFd(content, fd);
TEMP_FAILURE_RETRY(close(fd));
close(fd);
return result || CleanUpAfterFailedWrite(path);
}