Hi Joel, sorry for late response. I was too busy with other things. Again, I have very limited time to work on this. We are looking into
other changes in reflection that may or may not change and/or supersede this work.
Could you please share the details? I'm just curious.
I think this would be a fine contribution to get in to 9, a good baseline to build further work upon, but the scope has to be limited. Given the risk-reward here my bar for accepting this is quite high.
I'm looking forward to a rewritten test and some coverage information.
I have changed the test TestMethodReflectValueOf as you suggested and I have changed the summary of both tests too. You will find the changes in the attached patch. Here is the new webrev: https://db.tt/wQBLVceA And here is the coverage report in the HTML format: https://db.tt/JTZjpnMM Let me know if you need the coverage report in the text format. Best regards, Andrej Golovnin