Is this loop actually required? For mapping properties we only invoke method with a single parameter (methods with several parameters are only supported as public "entry point" for the user of a mapper).

Btw. I like the visitor approach. I'm still in the process of digging through everything, but this looks very good :)