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 authored and arch1t3cht committed Jan 21, 2025
1 parent 22b03bc commit 9b644d3
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 9b644d3

Please sign in to comment.