io/FileOutputStream: simplify OpenTempFile() call

This commit is contained in:
Max Kellermann 2022-05-13 13:21:01 +02:00
parent bd3e096411
commit 2670bbdcc8
1 changed files with 15 additions and 17 deletions

View File

@ -190,13 +190,16 @@ OpenTempFile(FileDescriptor directory_fd,
#endif /* HAVE_O_TMPFILE */
inline void
FileOutputStream::OpenCreate(bool visible)
FileOutputStream::OpenCreate([[maybe_unused]] bool visible)
{
#ifdef HAVE_O_TMPFILE
/* try Linux's O_TMPFILE first */
is_tmpfile = !visible && OpenTempFile(directory_fd, fd, GetPath());
if (!is_tmpfile) {
if (!visible && OpenTempFile(directory_fd, fd, GetPath())) {
is_tmpfile = true;
return;
}
#endif
/* fall back to plain POSIX */
if (!fd.Open(
#ifdef __linux__
@ -207,11 +210,6 @@ FileOutputStream::OpenCreate(bool visible)
0666))
throw FormatErrno("Failed to create %s",
GetPath().c_str());
#ifdef HAVE_O_TMPFILE
}
#else
(void)visible;
#endif
}
inline void