diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-02-20 23:06:59 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-02-22 23:05:14 +0100 |
| commit | b29a0f4bc67f6f8cc84c3d88cdbb9582e35d3fed (patch) | |
| tree | 8e518d1300e96ea7385c84a490015bf54ad998d6 /src/wx/email_dialog.cc | |
| parent | 674b74173d2d0ec8e178fa0938a4c48c2863c38b (diff) | |
Return std::vector instead of boost::optional from the EditableList dialog.
It's a bit clumsy, as returning more than one thing only makes sense
when adding (not when editing), but allowing both optional and vector
with template voodoo seems awkward (at least with C++11).
Diffstat (limited to 'src/wx/email_dialog.cc')
| -rw-r--r-- | src/wx/email_dialog.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/wx/email_dialog.cc b/src/wx/email_dialog.cc index 50fa66dcc..4f33f9a5a 100644 --- a/src/wx/email_dialog.cc +++ b/src/wx/email_dialog.cc @@ -23,9 +23,9 @@ #include "wx_util.h" -using std::string; using std::shared_ptr; -using boost::optional; +using std::string; +using std::vector; EmailDialog::EmailDialog (wxWindow* parent) @@ -47,8 +47,8 @@ EmailDialog::set (string address) } -optional<string> -EmailDialog::get () const +vector<string> +EmailDialog::get() const { auto s = wx_to_std (_email->GetValue ()); if (s.empty()) { @@ -56,5 +56,5 @@ EmailDialog::get () const return {}; } - return s; + return { s }; } |