[Mono-docs-list] [PATCH] Generics Support for monodoc/tools

[Joshua Tauberer]

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