Skip to content

Commit

Permalink
Add missing include header for wxTextValidator
Browse files Browse the repository at this point in the history
The use of wxTextValidator was introduced by upstream commit
5eb5d1a, but w/o the proper
include header, i.e. '#include <wx/valtext.h>'.

Error reported on Gentoo GNU/Linux amd64:

../aegisub-9999/src/dialog_dummy_video.cpp: In constructor ‘{anonymous}::DialogDummyVideo::DialogDummyVideo(wxWindow*)’:
../aegisub-9999/src/dialog_dummy_video.cpp:120:9: error: ‘wxTextValidator’ was not declared in this scope; did you mean ‘wxValidator’?
  120 |         wxTextValidator fpsVal(wxFILTER_INCLUDE_CHAR_LIST, &fps);
      |         ^~~~~~~~~~~~~~~
      |         wxValidator
../aegisub-9999/src/dialog_dummy_video.cpp:121:9: error: ‘fpsVal’ was not declared in this scope
  121 |         fpsVal.SetCharIncludes("0123456789./");
      |         ^~~~~~

References:
 - https://docs.wxwidgets.org/3.2.5/classwx_text_validator.html
  • Loading branch information
mehw committed Jan 21, 2025
1 parent eb2e503 commit c163b70
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/dialog_dummy_video.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
#include <wx/string.h>
#include <wx/textctrl.h>
#include <wx/valgen.h>
#include <wx/valtext.h>

namespace {
struct DialogDummyVideo {
Expand Down

0 comments on commit c163b70

Please sign in to comment.