doctest failure in cpp_classes.pyx

[craigcitro]

I ran into the following doctest failure with the current cython-devel (i.e. after the C++ merge):

...
AssertionError: None != u"44:33: Invalid conversion from 'Python object' to 'int'"

This comes down to not falling into the "right" case in the best_match function in Cython/Compiler/PyrexTypes.py. The attached patch cleans this function up a bit, and adds a few comments. I'm happy to commit, but wouldn't mind one more set of eyes first.

(Even with this patch, there's still one issue I'm confused about: int.assignable_from(object) is returning False. My off-the-cuff guess would have been the opposite; if so, someone should open another ticket and assign it to me.)

[robertwb]

The reason to use func_type rather than func.type is that func.type may be a pointer to a function, rather than the function itself.

[craigcitro]

Ah, I missed one or two occurrences of func_type below. I just passed it down from the top for loop, and pushed the change:

 http://hg.cython.org/cython-devel/rev/c5e63765abd1

[craigcitro]

... and the obligatory typo:

 http://hg.cython.org/cython-devel/rev/afd87a2d8004

[robertwb]

This was fixed and merged.