Details
-
Suggestion
-
Resolution: Unresolved
-
P3: Somewhat important
-
4.7.0
-
None
Description
QTextFormat: A vector is used to store the properties, however it should be confirmed if this is the best thing to be using. According to what I have been told the reason it uses a vector is to ensure that the order that the properties have been set is respected. However the code does not seem to indicate that this is happening currently and because the fact a vector is used the == operator will fail if the properties are added in a different order.
Therefore it should be confirmed that a vector is the correct thing to use here for this case, and subsequently check that it does indeed respect the order if that is the intention of it.