Fix two things -Wconditional-uninitialized warned about

Rather minor stuff:
* Initliaze first case to 0.
* The other (saved_errno), move the error output it is for
  in-scope and not need at all.
This commit is contained in:
Aaron Gyes 2016-07-30 03:08:57 -07:00
parent 3a08a24259
commit acfd380176
2 changed files with 6 additions and 7 deletions

View file

@ -189,7 +189,7 @@ static wcstring str2wcs_internal(const char *in, const size_t in_len) {
mbstate_t state = {};
while (in_pos < in_len) {
bool use_encode_direct = false;
size_t ret;
size_t ret = 0;
wchar_t wc = 0;
if ((in[in_pos] & 0xF8) == 0xF8) {

View file

@ -570,7 +570,6 @@ bool env_universal_t::open_temporary_file(const wcstring &directory, wcstring *o
assert(!string_suffixes_string(L"/", directory));
bool success = false;
int saved_errno;
const wcstring tmp_name_template = directory + L"/fishd.tmp.XXXXXX";
wcstring tmp_name;
@ -587,15 +586,15 @@ bool env_universal_t::open_temporary_file(const wcstring &directory, wcstring *o
}
#endif
saved_errno = errno;
success = result_fd != -1;
*out_fd = result_fd;
*out_path = str2wcstring(narrow_str);
free(narrow_str);
if (!success && attempt == 9) {
report_error(errno, L"Unable to open temporary file '%s'", narrow_str);
}
if (!success) {
report_error(saved_errno, L"Unable to open file '%ls'", out_path->c_str());
*out_path = str2wcstring(narrow_str);
free(narrow_str);
}
return success;
}