Code Review: ProGenFix


tfpt review “/shelveset:ProGenFix;REDMOND\tomat”

DLR, Python, Ruby.

Fixes Ruby calls to protected generic methods.
Adds CompilerHelpers.IsProtected() as an alias for IsFamily ||



Changes look good overall. I’m not sure that Type.IsProtected() is an
intuitive name; maybe Type.IsNested() would be clearer?


You mean IsNestedProteced()? “Nested” feels superfluous since only
nested types could be protected.



Ah, I see. Still, something strikes me as odd about the name, but I
don’t feel strongly about it.