[PATCH] D42850: [docs] Add guidance on duplicating doc comments to CodingStandards

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 5 09:54:42 PST 2018


echristo added inline comments.


================
Comment at: docs/CodingStandards.rst:324
 ``\brief`` adds visual clutter.  Put detailed discussion into separate
-paragraphs.
+paragraphs. Avoid duplicating documentation for overridden methods. When
+extending documentation from the superclass, the ``\copydoc name`` command can
----------------
reames wrote:
> I'd suggest reframing this to focus on what should you document for the override version.  i.e. When documenting override methods, highlight the differences with parent implementation.  Instead of duplicating the existing documentation, refer to it if needed.  If the documentation for the overridden method is sufficient to document the override's behaviour, omit documentation of the override.
+1


Repository:
  rL LLVM

https://reviews.llvm.org/D42850





More information about the llvm-commits mailing list