Hey, Jon. Very nice.
I won't have a chance to look at the patch for a few days but I had two comments: > The one breaking change to monodocer.exe is that '+' is no longer used > for nested types, but '.' is used instead. That is: > > public class Outer { public class Inner {}} > > generates the file Outer.Inner.xml, while as before it would generate > Outer+Inner.xml. This change is necessary for monodocs2html.exe, as > there is no way from a <see cref="T:Namespace.Outer.Inner" /> reference > to tell which of those strings is the namespace, the outer class, and > the inner class (at least, no easy way that I can see), so removing the > '+' makes monodocs2html.exe easier to implement. I don't see how that helps, since the full file path is Namespace/Outer+Inner.xml --- which means it already needs to know which period ends the namespace so it knows which directory has the file, and if it knows that, then it knows the next period must be a nested class period. > (It also is ugly-as-sin, containing joyfully recursive XSLT calls to > convert "Foo<A,B>.Nested<C>" into "Foo`2.Nested`1" constructs. Lots of > fun. Really. Stop looking at me like that!) Fun, eh? -- - Joshua Tauberer http://razor.occams.info "Strike up the klezmer and start acting like a man. You're about to have a truth-mitzvah." -- The Colbert Report _______________________________________________ Mono-docs-list maillist - Mono-docs-list@lists.ximian.com http://lists.ximian.com/mailman/listinfo/mono-docs-list