[C++ PATCH]: Bug 1617

[Mark Mitchell]

Wow.  What a mess.

`T*' is more specialized that `T const &'?  I see, because a `T
const&' is pretty much like a `T'?  Is that the thinking.

!       fate = more_specialized (TREE_VALUE (champ), TREE_VALUE (fn), 99999);

What's with the 99999?  Surely we need some name for this magic value?
What does 99999 signify?

+          pashing, or we can bash the ARG too. The latter, though seemingly

bashing, I think you mean.  And, I personally don't like that term.
arg-bashing sounds not very PC. :-) Seriously, I think we should avoid
slang in the compiler comments; it's too hard to process for
non-native speakers, or for native speakers who aren't "real hackers".

So, let's rework the comment to explain what process you're talking
about.

Once we resolve the 99999 thing, the patch is OK.

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com