public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* Thoughts on doxygen for internal documentation
@ 2003-01-15 23:48 Diego Novillo
  2003-01-16 10:47 ` Phil Edwards
                   ` (2 more replies)
  0 siblings, 3 replies; 26+ messages in thread
From: Diego Novillo @ 2003-01-15 23:48 UTC (permalink / raw)
  To: gcc

I have tried using doxygen to document the tree-ssa API.  Given
the prevalent documentation style in GCC, I find some of the
markers rather intrusive.

For instance, it insists that documentation for arguments should
be preceded by @param.  It also doesn't understand that we refer
to the argument names in capitals.  This breaks the flow of the
comment and makes it harder to read.

I don't mind the @brief marker that it wants for adding one liner
descriptions in the TOC.  But again, I'd rather not add it.  What
I find nice are the @file markers for describing the file.

What is really useful about doxygen is all the cross referencing
and html pages it builds.  Also, we don't really need to mark up
everything for doxygen to be useful.  It's enough that we delimit
the function comments with /** */.

For the tree-ssa branch, I'd like to propose the following:

1- When writing the preceding documentation for a function, data
   structure or variable, delimit it with /**  */.  This way,
   doxygen will add it when generating documentation.


2- For new files use the @file and @brief markers to provide a
   brief description for the file.

No other markers should be required.  It would be nice if the
rest of the compiler used the /** */ comments, but for now I'm
happy if we just do it for the tree-ssa files.


Thoughts?   Diego.

^ permalink raw reply	[flat|nested] 26+ messages in thread
* Re: Thoughts on doxygen for internal documentation
@ 2003-01-18 16:17 Robert Dewar
  0 siblings, 0 replies; 26+ messages in thread
From: Robert Dewar @ 2003-01-18 16:17 UTC (permalink / raw)
  To: Theodore.Papadopoulo, law; +Cc: dnovillo, gcc

> As an example, I always wondered why the rule of 80 columns is still
> that strict. IMHO often code can be more readable using longer lines
> (still with a reasonnable limit, just an higher one).  Yes I know, some
> people still have a very old 80 columns VT100, but those cannot be 
> that many ? Indeed, mail reader sometimes have this 80 limit constraints,
> but then problems with line wrapping by mailers happen also with 80 
> columns... So why keeping this constraints and not updating it to 
> something more sensible such as eg 132 colums.

It is really useful to maintain the 80 column rule, otherwise you get
into a mess in using screen real estate, or messing with horizontal 
scrolling in an editor.

^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2003-01-18 17:03 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-01-15 23:48 Thoughts on doxygen for internal documentation Diego Novillo
2003-01-16 10:47 ` Phil Edwards
2003-01-16 18:12 ` law
2003-01-16 18:14   ` Diego Novillo
2003-01-16 18:16     ` law
2003-01-16 18:18       ` Theodore Papadopoulo
2003-01-16 18:57         ` Gabriel Dos Reis
2003-01-16 19:11         ` law
2003-01-16 19:53           ` Andrew Haley
2003-01-16 23:36           ` Toon Moene
2003-01-16 19:18         ` Joel Sherrill
2003-01-16 19:35         ` Marc Espie
2003-01-16 19:37         ` Andrew Haley
2003-01-16 19:57           ` Theodore Papadopoulo
2003-01-17  0:26         ` Daniel Egger
2003-01-17 16:15           ` Daniel Berlin
2003-01-19  5:09             ` Daniel Egger
2003-01-16 18:20       ` Diego Novillo
2003-01-16 18:27         ` Gabriel Dos Reis
2003-01-16 18:29         ` law
2003-01-16 18:25       ` Daniel Berlin
2003-01-17  4:00     ` Kurt Wall
2003-01-16 20:03   ` Joseph S. Myers
2003-01-17 11:05 ` Tony Finch
2003-01-18 22:21   ` Daniel Egger
2003-01-18 16:17 Robert Dewar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).