maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Fix some rather confusing indentation and control flow in the errno 

printing routine. This is made harder to see due to the surprising
formatting, inconsistent brace usage, and repeated conditions that all
test the same thing.

The only "consequence" of this bug is re-assigning 'str' to an empty
string when computing the error string for an error number of 0 in the
event of a non-GNU strerror_r routine. So, nothing to see here other
than cleanup. It did help me find PR17055 in clang-format though.

llvm-svn: 189734
This commit is contained in:
Chandler Carruth 2013-09-02 05:55:10 +00:00
parent 6e0520e9d9
commit 14aae04029
1 changed files with 14 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
llvm/lib/Support/Errno.cpp
View File

@ -39,28 +39,27 @@ std::string StrError(int errnum) {
char buffer[MaxErrStrLen];
buffer[0] = '\0';
std::string str;
if (errnum == 0)
return str;
#ifdef HAVE_STRERROR_R
// strerror_r is thread-safe.
if (errnum)
# if defined(__GLIBC__) && defined(_GNU_SOURCE)
// glibc defines its own incompatible version of strerror_r
// which may not use the buffer supplied.
str = strerror_r(errnum,buffer,MaxErrStrLen-1);
# else
strerror_r(errnum,buffer,MaxErrStrLen-1);
str = buffer;
# endif
#if defined(__GLIBC__) && defined(_GNU_SOURCE)
// glibc defines its own incompatible version of strerror_r
// which may not use the buffer supplied.
str = strerror_r(errnum, buffer, MaxErrStrLen - 1);
#else
strerror_r(errnum, buffer, MaxErrStrLen - 1);
str = buffer;
#endif
#elif HAVE_DECL_STRERROR_S // "Windows Secure API"
if (errnum) {
strerror_s(buffer, MaxErrStrLen - 1, errnum);
str = buffer;
}
strerror_s(buffer, MaxErrStrLen - 1, errnum);
str = buffer;
#elif defined(HAVE_STRERROR)
// Copy the thread un-safe result of strerror into
// the buffer as fast as possible to minimize impact
// of collision of strerror in multiple threads.
if (errnum)
str = strerror(errnum);
str = strerror(errnum);
#else
// Strange that this system doesn't even have strerror
// but, oh well, just use a generic message