[mono-api-html] Make --ignore-changes-virtual less lenient.
authorJonathan Pryor <jonpryor@vt.edu>
Thu, 13 Nov 2014 18:02:34 +0000 (13:02 -0500)
committerJonathan Pryor <jonpryor@vt.edu>
Thu, 13 Nov 2014 18:36:45 +0000 (13:36 -0500)
commit09d16b7265dc15d9ebe2d1331e1a41bdaaa68ef5
tree72ef0ccb64ac4c21c282225b26880571cafa867c
parent3d293f5a719db1fee145827e8f2f3ef70e2c1356
[mono-api-html] Make --ignore-changes-virtual less lenient.

As mentioned in commit 789517f8, the intentino of
`mono-api-html --ignore-changes-virtual` is so that when a method is
changed from non-virtual to virtual or override, or from virtual to
override, a "Removed member" message isn't triggered.

However, the previous implementation was too lenient. While *adding*
virtual to the method is fine, *removing* is NOT fine, as that would
be an API breaking change (e.g. all classes which attempt to override
a previously virtual method could no longer do so).

Make the virtual check stricter so that virtual/override to
non-virtual changes are not ignored.
mcs/tools/corcompare/mono-api-html/ConstructorComparer.cs
mcs/tools/corcompare/mono-api-html/MemberComparer.cs