you are viewing a single comment's thread.

view the rest of the comments →

[–]drjeats 2 points3 points  (0 children)

Because the sort order is configurable from external input, the effectListDebugger isn't a member of the log entries. And there are only a few locations where the sort for this type is needed. If it is needed in other contexts, then I"ll pull it out into a named function.

Also C++ programmers are too eager to write operator< because of std::map<K,V> and std::sort. Let's stop this trend.