InputConfigDialog: Don't show "..." for complicated expressions
Just show the actual expression. We need to do a bit of mangling here as wx has no way to turn off mnemonics parsing, so do that as well.
This commit is contained in:
parent
eb153cfded
commit
d5983b587e
|
@ -240,10 +240,9 @@ void GamepadPage::UpdateGUI()
|
|||
, e = (*g)->control_buttons.end();
|
||||
for (; i!=e; ++i) {
|
||||
ControllerInterface::ControlReference *r = (*i)->control_reference;
|
||||
if (r->IsComplicated())
|
||||
(*i)->SetLabel("...");
|
||||
else
|
||||
(*i)->SetLabel(StrToWxStr((*i)->control_reference->expression));
|
||||
wxString expr = StrToWxStr((*i)->control_reference->expression);
|
||||
expr.Replace("&", "&&");
|
||||
(*i)->SetLabel(expr);
|
||||
}
|
||||
|
||||
// cboxes
|
||||
|
|
|
@ -73,13 +73,6 @@ public:
|
|||
return 0;
|
||||
}
|
||||
|
||||
bool IsComplicated() {
|
||||
if (parsed_expression)
|
||||
return parsed_expression->is_complicated;
|
||||
else
|
||||
return false;
|
||||
}
|
||||
|
||||
protected:
|
||||
ControlReference(const bool _is_input) : range(1), is_input(_is_input), parsed_expression(NULL) {}
|
||||
ciface::ExpressionParser::Expression *parsed_expression;
|
||||
|
|
|
@ -210,7 +210,6 @@ public:
|
|||
virtual ControlState GetValue() { return 0; }
|
||||
virtual void SetValue(ControlState state) {}
|
||||
virtual int CountNumControls() { return 0; }
|
||||
virtual bool IsComplicated() { return false; }
|
||||
virtual operator std::string() { return ""; }
|
||||
};
|
||||
|
||||
|
@ -237,11 +236,6 @@ public:
|
|||
return 1;
|
||||
}
|
||||
|
||||
virtual bool IsComplicated()
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
virtual operator std::string()
|
||||
{
|
||||
return "`" + (std::string)qualifier + "`";
|
||||
|
@ -293,11 +287,6 @@ public:
|
|||
return lhs->CountNumControls() + rhs->CountNumControls();
|
||||
}
|
||||
|
||||
virtual bool IsComplicated()
|
||||
{
|
||||
return true;
|
||||
}
|
||||
|
||||
virtual operator std::string()
|
||||
{
|
||||
return OpName(op) + "(" + (std::string)(*lhs) + ", " + (std::string)(*rhs) + ")";
|
||||
|
@ -345,11 +334,6 @@ public:
|
|||
return inner->CountNumControls();
|
||||
}
|
||||
|
||||
virtual bool IsComplicated()
|
||||
{
|
||||
return true;
|
||||
}
|
||||
|
||||
virtual operator std::string()
|
||||
{
|
||||
return OpName(op) + "(" + (std::string)(*inner) + ")";
|
||||
|
@ -539,7 +523,6 @@ Expression::Expression(ExpressionNode *node_)
|
|||
{
|
||||
node = node_;
|
||||
num_controls = node->CountNumControls();
|
||||
is_complicated = node->IsComplicated();
|
||||
}
|
||||
|
||||
Expression::~Expression()
|
||||
|
|
|
@ -51,7 +51,6 @@ public:
|
|||
ControlState GetValue();
|
||||
void SetValue (ControlState state);
|
||||
int num_controls;
|
||||
bool is_complicated;
|
||||
ExpressionNode *node;
|
||||
};
|
||||
|
||||
|
|
Loading…
Reference in New Issue