DescriptionFixing invalid operator< implementation.
It was possible that "A < B" and "B < A" both evaluated to true.
This manifested as an assert on Windows, and a memory leak on Linux.
Note that the concept of "less than" is meaningless for this object.
The operator is only needed so the object can be used as a key in an
std::map.
BUG=webrtc:6068
R=honghaiz@webrtc.org, kjellander@webrtc.org, skvlad@webrtc.org
Committed: https://crrev.com/734262c765dbd13b34da1d016c82c7d3d2c4c50c
Cr-Commit-Position: refs/heads/master@{#13598}
Patch Set 1 #
Total comments: 4
Patch Set 2 : Using std::tie, added unit test. #Patch Set 3 : Fixing comment. #Patch Set 4 : . #
Messages
Total messages: 20 (7 generated)
|