[Mono-docs-list] More monodocer

[Jonathan Pryor]

On Tue, 2006-10-10 at 07:37 -0400, Joshua Tauberer wrote:
> Jonathan Pryor wrote:
> > I see that you committed your changes, and I'm seeing changes that I
> > don't understand.
> > 
> > For example, after running the check-monodocer test I see:
> > 
> > -  <Base>
> > -    <BaseTypeName>System.Object</BaseTypeName>
> > -  </Base>
> > +  <Base />
> > +  <BaseTypeName>System.Object</BaseTypeName>
> 
> That's weird.  Obviously I goofed something (in my second commit
> probably)...

Yes, your 2nd commit caused it.

A fix is to change line 699:

-WriteElementText(root, "Base/BaseTypeName", basetypename);
+WriteElementText(basenode, "BaseTypeName", basetypename);

However, I'm not sure *why* this would be necessary, since
WriteElementText() calls WriteElement(), which splits the string along
'/', so the original *should* be correct (as SelectSingleNode("Base")
should return basenode, allowing BaseTypeName to be created underneath
Base).  Very confusing.

Another problem is that the position of /Type/TypeParameters has changed
from DocTest/en.expected -- it used to be after the /Type/Docs node, and
now it's before the /Type/Base node.

The *real* problem is that the position of /Type/TypeParameters changes
between the initial documentation generation and on update.  Initial
creation places it before /Type/Base, while an update places it last.
This creates spurious differences.

 - Jon