]> https://gitweb.dealii.org/ - dealii.git/commitdiff
update documentation
authorMatthias Maier <tamiko@43-1.org>
Sat, 16 Jun 2018 03:49:07 +0000 (22:49 -0500)
committerMatthias Maier <tamiko@43-1.org>
Sat, 16 Jun 2018 04:46:10 +0000 (23:46 -0500)
contrib/utilities/indent
contrib/utilities/indent-branch
doc/doxygen/headers/coding_conventions.h

index 54e2ced7b6305f623e5502a24763b5ec8fa1c8a7..afa726ce428fc9ef1f2ae51bc9ed76f5b8d03929 100755 (executable)
 ## ---------------------------------------------------------------------
 
 #
-# This script indents all source files of deal.II according to our
-# usual code formatting standards. It is used to ensure that our
-# code base looks uniform, as uniformity helps make code easier to
-# read.
+# This script indents all source files of deal.II according to our usual
+# code formatting standards. It is used to ensure that our code base looks
+# uniform, as uniformity helps make code easier to read.
 #
 # While we're already touching every file, this script also makes
-# sure we set permissions correctly
+# sure we set permissions correctly and checks for correct unix-style line
+# endings.
 #
 # The script needs to be executed as
-#   ./contrib/utilities/indent
+#   ./contrib/utilities/indent
 # from the top-level directory of the source tree, or via
-#   make indent
+#   make indent
 # from a build directory.
 #
+# Note: If the script is invoked with REPORT_ONLY=true set,
+#   $ REPORT_ONLY=true ./contrib/utilities/indent
+# or,
+#   $ make REPORT_ONLY=true indent
+# then indentation errors will only be reported without any actual file
+# changes.
+#
 
 if [ ! -f contrib/utilities/indent_common.sh ]; then
   echo "*** This script must be run from the top-level directory of deal.II."
index 46f1412d580794739c3bb0e235b32d9a870c193a..e2bed13546257ec3f223bbfc070d36f89d8c4568 100755 (executable)
 #
 # This script does the same thing as contrib/utilities/indent but only
 # reformats files which have changed (or have been added but neither
-# staged/commited) since the last merge commit to the master branch. It
-# should be run as
-#
-#    make indent-branch
+# staged/commited) since the last merge commit to the master branch.
 #
+# The script needs to be executed as
+#   $ ./contrib/utilities/indent-branch
+# from the top-level directory of the source tree, or via
+#   $ make indent-branch
 # from a build directory.
 #
+# Note: If the script is invoked with REPORT_ONLY=true set,
+#   $ REPORT_ONLY=true ./contrib/utilities/indent-branch
+# or,
+#   $ make REPORT_ONLY=true indent-branch
+# then indentation errors will only be reported without any actual file
+# changes.
+#
 
 if [ ! -f contrib/utilities/indent_common.sh ]; then
   echo "*** This script must be run from the top-level directory of deal.II."
index 7cecc12ab0d2535af1f9c9a9e44f03e5bc8d437d..2cedd9cd3b3daaa210ba3f90bbef6c11bf95dba2 100644 (file)
@@ -50,14 +50,16 @@ on each of your files. This will make sure indentation is conforming to the
 style guidelines outlined in this page. Alternatively, you can run
 <code>
 <pre>
-  make indent
+  make indent-branch
 </pre>
 </code>
 in whatever directory you set up the library to be compiled in to indent all
-source files. If you want to make sure that the indenting is correct for all
-your commits, you might want to set up a pre-commit hook. One way to do so,
-is to copy ${SOURCE_DIR}/contrib/scripts/pre-commit-clang-format to
-${SOURCE_DIR}/.git/hooks/pre-commit and make sure it is executable.
+source files that have been changed recently. If you want to make sure that
+the indenting is correct for all your commits, you might want to set up a
+pre-commit hook. One way to do so, is to copy
+<code>${SOURCE_DIR}/contrib/scripts/pre-commit-clang-format</code> to
+<code>${SOURCE_DIR}/.git/hooks/pre-commit</code> and make sure it is
+executable.
 </p>
 
 <h3>Style issues</h3>
@@ -67,8 +69,8 @@ ${SOURCE_DIR}/.git/hooks/pre-commit and make sure it is executable.
   degrees of freedom, etc) should start with <code>n_*</code>. Example:
   SparsityPattern::n_nonzero_entries().</li>
 
-<li> %Function which set a bit or flag should start with <code>set_*</code>;
-  functions which clear bits of flags should be named <code>clear_*</code>.
+<li> %Functions which set a bit or flag should start with <code>set_*</code>;
+  functions which clear bits or flags should be named <code>clear_*</code>.
   Example: CellIterator::set_refine_flag().</li>
 
 <li> Traditional logical operators should be used instead of their English

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.