[mdoc] Follow up fix for member and type removal.
authorJoel Martinez <joelmartinez@gmail.com>
Fri, 4 Sep 2015 21:27:10 +0000 (17:27 -0400)
committerJoel Martinez <joelmartinez@gmail.com>
Sat, 5 Sep 2015 00:10:24 +0000 (20:10 -0400)
commitb94920854fdab174f9e0b90a7919b19b8fefa256
tree5ba722360660dfb9c4c0a534f5c8a269cbbec628
parent09b97cba2e7c2d9c68830d55ff3d99c5f36eb439
[mdoc] Follow up fix for member and type removal.
In supplement of the changes submitted in 09b97cba2e7c2d9c68830d55ff3d99c5f36eb439, there were
some additional edge cases that were not originally considered, and so this patch resolves these
additional issues. In particular, there was a bug in the `RemoveApiStyle` method that wasn't
checking the value of the `apistyle` attribute before removing it. This caused members to be
stripped of apistyle inadvertently.

There was an issue in the type removal code that caused
too many files to be removed. This patch adds additional checks to make sure that a file isn't
going to be removed if it contains members in the other style that shoulnd't be removed

Finally, there was an issue that arose after the aforementioned changes in the `OrderTypeAttributes`
method. It was somewhat counterintuitive, but it wasn't allowing the reordering of attributes
with the `.Remove` method (throwing an exception). Not entirely sure why this was happening,
but using the `.RemoveNamedItem` method worked around the issue.
mcs/tools/mdoc/Mono.Documentation/monodocer.cs
mcs/tools/mdoc/Test/en.expected-dropns-delete/MyFramework.MyNamespace/WillDelete.xml.remove