James Turner wrote: > On 31 Aug 2008, at 11:58, Stefan C. Müller wrote: > >> OK, here's the patch. >> But we need a total of 3 overloaded operator(). Not realy nice to >> look at. >> Anyway, I'm happy with both variants. > > As you say, not ideal, but I'm still working on this code, so a little > bit of uglyness, I can live with for the moment - main thing is to > keep MSVC builds happy. > > If someone with commit access could apply, that's be great.
If you're going to define a comparison operator for use with the STL that takes two different typed arguments, you need to define the three versions because you can't be sure which order any given STL implementation will use. I guess you're not using map::lower_bound because you want to support an arbitrary ordering different from the map? What will the lower bound result mean in that case? Tim ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Flightgear-devel mailing list Flightgear-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/flightgear-devel