ilovepi wrote: > > Do we know what other documentation generators output? I don't know that > > I've seen the underlying type displayed unless its not the default (e. g. > > the case you'd expect based on language rules). > > Ah, you're right. I thought that this case was explicitly declared with an > `int` type. I think `enum class` gets assigned an integral type by default, > hence `BaseType` being serialized here but not with the other cases.
Doxygen at least seems to omit the type if its the default: https://clang.llvm.org/doxygen/namespaceclang.html#a75ddc0e87edf9dee590675f81303cbba. I'm not sure if they do that if its specified in source but also is the default (we'd have to look more to be sure). I think we should take a pass over the output, add a couple extra tests (can be standalone e.g. outside of the basic project), and then determine if 1) we want to always print the underlying type or omit it and 2) asses how difficult that is to support if we want it. If its going to be a lot of extra work, we may just want to always print it out, but I'm not sure that's exactly the right call, since it could give folks the impression that its specified, when it isn't. Likely we'll end up with a new FIXME or TODO with some case still unhandled/unclear. https://github.com/llvm/llvm-project/pull/181347 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
