Skip to content

Commit

Permalink
log/windows: Fix missing class member initializer
Browse files Browse the repository at this point in the history
This had the confusing side-effect of making some code in wesnoth.cpp
believe it was safe to prompt the user to press Enter when NOT using
--wconsole on my system, because the flag ended up with a true value by
default.

This is why you don't leave integral-type variables uninitialized, ever.
  • Loading branch information
irydacea committed Oct 26, 2020
1 parent 4abeef2 commit cb867c0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
2 changes: 2 additions & 0 deletions changelog.md
Expand Up @@ -2,6 +2,8 @@
### Language and i18n
* Updated translations: British English, Catalan, French, Japanese, Polish,
Portuguese (Brazil)
### Miscellaneous and Bug Fixes
* Fixed a rare issue on Windows that could result in wesnoth.exe sticking around waiting for console input after encountering an error despite not being launched with the `--wconsole` option.

## Version 1.14.14
### Add-ons client
Expand Down
1 change: 1 addition & 0 deletions src/log_windows.cpp
Expand Up @@ -284,6 +284,7 @@ log_file_manager::log_file_manager(bool native_console)
: fn_(unique_log_filename())
, cur_path_()
, use_wincon_(console_attached())
, created_wincon_(false)
{
DBG_LS << "Early init message\n";

Expand Down

0 comments on commit cb867c0

Please sign in to comment.