case '|': // | or ||
case '|': // | or ||
if (it[1] == '*')
##N#ew###New Style auto return I still prefer the old style of function declaration over the new style.
case '-': // - (unary or binary)
case '-': // - (unary or binary)
if (res.empty())
{
// STUFF
break
}
else
{
// STUFF
break;
}
Personally iI would havemove the breaks outside the if statement. So it is clear that the case statement isdoes have a break.
##Changes that may help: ###Limit of 16 parameters:
Number params[16];
Number params[16];
// STUFF
operands.push(func(params));
break;
##New Stuff: ###Not seen this before callbacks.emplace( std::piecewise_construct, // Will have to go read about this. std::forward_as_tuple(name), std::forward_as_tuple(function) ); ###Would using move semantics help for future
callbacks.emplace(
output std::piecewise_construct, // Will have to go read about this.push
std::forward_as_tuple(operations.topname),
std::forward_as_tuple(function)
);
###Would using move semantics help for future
output.push(operations.top());