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 _______________________________________________ Mono-docs-list maillist - Mono-docs-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-docs-list