In InstanceKlass::mark_newly_obsolete_methods which is called during RedefineClasses,

[~sspitsyn] asks why the check emcp_method_count != 0 is needed at the line:
4005 if (emcp_method_count != 0 && obsolete_method_count != 0 &&

Suppose, the emcp_method_count is equal to 0.
Then do we really want to skip marking all the methods of this InstanceKlass as obsolete?

It doesn't look right to me either, and there should be a test (can adapt RedefineRunningMethods.java)