refactor: units ordering do need type_name (it also caused worse ordering on text output)

This commit is contained in:
Mateusz Pusz
2022-12-27 10:08:59 +01:00
parent ff14b49b68
commit 210f401416

View File

@@ -25,7 +25,6 @@
#include <units/bits/algorithm.h>
#include <units/bits/expression_template.h>
#include <units/bits/external/fixed_string.h>
#include <units/bits/external/type_name.h>
#include <units/bits/external/type_traits.h>
#include <units/bits/magnitude.h>
#include <units/bits/ratio.h>
@@ -526,7 +525,7 @@ template<Unit Lhs, Unit Rhs>
{
if ((is_derived_from_specialization_of_constant_unit<Lhs> && is_derived_from_specialization_of_constant_unit<Rhs>) ||
(!is_derived_from_specialization_of_constant_unit<Lhs> && !is_derived_from_specialization_of_constant_unit<Rhs>))
return type_name<Lhs>() < type_name<Rhs>();
return Lhs::symbol < Rhs::symbol;
else
// put constants at the front of units list in the expression
return is_derived_from_specialization_of_constant_unit<Lhs>;