[PATCH 00/21] Documentation improvements

Matthias Clasen matthias.clasen at gmail.com
Mon Apr 1 09:02:21 PDT 2013


On Mon, Apr 1, 2013 at 10:43 AM, Jason Ekstrand <jason at jlekstrand.net> wrote:
> Matthias,
> First, I would like to apologize for a) being a bit harsh and
> overbearing in my comments and b) getting too caught up in stylistic
> issues.  I really should have read through it for content first and
> just kept a tally for style things.  At some point here (probably not
> till at least the weekend) I will try to go through and do a thorough
> content-only review of the wayland.xml changes.  I will leave the
> style discussion here and we can figure out how we want to handle each
> of these issues in one place.  Disagreements about style really
> shouldn't be handled inline anyway.

No worries, I didn't take it personally. Stylistic editing and patch
review are not easy to marry. On some level, it would be best if krh
just did an editing pass and committed a version of the docs that he
is happy with. I was hoping my patches could serve as a basis for
that.


>>
>> For judging formatting questions like this, I recommend proof-reading
>> the generated docs. That's what I looked at when making those changes.
>
> Allow me to explain my perspective on this a bit more.  I am not only
> looking at the HTML generated by publican (as given on the webpage)
> but my own XML protocol parser that I have written to dump out Java
> classes for wayland-java generates JavaDoc comments for each
> event/request/interface as per the XML documentation flags.  I also
> think it would be a good idea to, at some point, make the C scanner
> dump out doxygen comments.  Later this week I'll try to get publican
> installed so I can understand the documentation pipeline a bit better.

Interesting. I didn't know that there are other consumers than what's
in the wayland repo.


>
> References to arguments or events/requests:
> One thing Matthias pulled out was some "@" characters before arguments
> or event/request names.  I agree that the "@" isn't really doing
> anything for us.  That said, I'm not sure we just want to get rid of
> it.  It would be nice if we had something some sort of flag that
> publican could turn into <function> and other things could turn into
> whatever type of reference it requires.  I realize going through and
> adding that would be a lot of work.  Some times this can be easily
> fixed by simply moving pieces of the doc comment to argument comments.

Yeah, what annoyed me about the @ was that it doesn't do anything, is
inconsistently used, and makes it into the formatted document. I had
thought about using <function> markup, or some other suitable docbook
tag. It does create some nicer output. But I didn't want to disrupt
the text with too much markup. So far, it is really nicely readable
even in the xml.


More information about the wayland-devel mailing list