Cleanup: Make UI tree-view item constructor explicit
Constructors that can be called with a single argument should be explicit, unless there's a reason not to, to avoid unintended constructions.
This commit is contained in:
parent
933215d6da
commit
e16bc136f9
|
@ -401,7 +401,7 @@ class BasicTreeViewItem : public AbstractTreeViewItem {
|
|||
using ActivateFn = std::function<void(BasicTreeViewItem &new_active)>;
|
||||
BIFIconID icon;
|
||||
|
||||
BasicTreeViewItem(StringRef label, BIFIconID icon = ICON_NONE);
|
||||
explicit BasicTreeViewItem(StringRef label, BIFIconID icon = ICON_NONE);
|
||||
|
||||
void build_row(uiLayout &row) override;
|
||||
void on_activate(ActivateFn fn);
|
||||
|
|
Loading…
Reference in New Issue