Probably pretty minor, but the feature string buildup in NyxstoneBuilder::build is pretty inefficient, needing to likely reallocate all the feature string in order to add the + or -, and needing an extra intermediary vector. All of it can be avoided by manually building up inside the final string directly.
Probably pretty minor, but the feature string buildup in
NyxstoneBuilder::build
is pretty inefficient, needing to likely reallocate all the feature string in order to add the+
or-
, and needing an extra intermediary vector. All of it can be avoided by manually building up inside the final string directly.