diff options
Diffstat (limited to 'llvm/docs/CodingStandards.rst')
| -rw-r--r-- | llvm/docs/CodingStandards.rst | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/llvm/docs/CodingStandards.rst b/llvm/docs/CodingStandards.rst index 8c8110d0332..498d76b04d8 100644 --- a/llvm/docs/CodingStandards.rst +++ b/llvm/docs/CodingStandards.rst @@ -247,8 +247,8 @@ tree. The standard header looks like this: //===----------------------------------------------------------------------===// /// /// \file - /// \brief This file contains the declaration of the Instruction class, which is - /// the base class for all of the VM instructions. + /// This file contains the declaration of the Instruction class, which is the + /// base class for all of the VM instructions. /// //===----------------------------------------------------------------------===// @@ -268,10 +268,10 @@ file is released under. This makes it perfectly clear what terms the source code can be distributed under and should not be modified in any way. The main body is a ``doxygen`` comment (identified by the ``///`` comment -marker instead of the usual ``//``) describing the purpose of the file. It -should have a ``\brief`` command that describes the file in one or two -sentences. Any additional information should be separated by a blank line. If -an algorithm is being implemented or something tricky is going on, a reference +marker instead of the usual ``//``) describing the purpose of the file. The +first sentence or a passage beginning with ``\brief`` is used as an abstract. +Any additional information should be separated by a blank line. If an +algorithm is being implemented or something tricky is going on, a reference to the paper where it is published should be included, as well as any notes or *gotchas* in the code to watch out for. @@ -320,10 +320,11 @@ Doxygen Use in Documentation Comments Use the ``\file`` command to turn the standard file header into a file-level comment. -Include descriptive ``\brief`` paragraphs for all public interfaces (public -classes, member and non-member functions). Explain API use and purpose in -``\brief`` paragraphs, don't just restate the information that can be inferred -from the API name. Put detailed discussion into separate paragraphs. +Include descriptive paragraphs for all public interfaces (public classes, +member and non-member functions). Don't just restate the information that can +be inferred from the API name. The first sentence or a paragraph beginning +with ``\brief`` is used as an abstract. Put detailed discussion into separate +paragraphs. To refer to parameter names inside a paragraph, use the ``\p name`` command. Don't use the ``\arg name`` command since it starts a new paragraph that @@ -343,8 +344,8 @@ A minimal documentation comment: .. code-block:: c++ - /// \brief Does foo and bar. - void fooBar(bool Baz); + /// Sets the xyzzy property to \p Baz. + void setXyzzy(bool Baz); A documentation comment that uses all Doxygen features in a preferred way: @@ -401,10 +402,10 @@ Correct: // In Something.h: - /// \brief An abstraction for some complicated thing. + /// An abstraction for some complicated thing. class Something { public: - /// \brief Does foo and bar. + /// Does foo and bar. void fooBar(); }; |

