lijunle / Vsxmd

VS XML documentation -> Markdown syntax.
MIT License
217 stars 52 forks source link

NullPointerException on cref as an element instead of as an attribute #93

Open ptsneves opened 2 years ago

ptsneves commented 2 years ago
<exception>
   <cref>ForbiddenValueConfigurationException</cref>
</exception>

I have a generated xml that has the above element that leads to ExceptionUnit Name property to throw null pointer. I am not sure if this a valid XML, but it was automatically generated so I assume it is? The docs at microsoft indeed do not mention a element inside but other similar elements allow for such constructions.

The problem pops up because the ExceptionUnit expects the attribute cref to populate the Name property. I guess there needs to be code to catch the case where the cref is

Also we should really document that when Vsxmd throws an exception it does not abort the build. This leads to a very confusing diagnosis of why the .md is not generated.