From 115e70a1b7cb5d097787ea0d07f7e903b9cb99bd Mon Sep 17 00:00:00 2001 From: "G. Paul Ziemba" Date: Tue, 13 Mar 2018 17:32:51 -0700 Subject: [PATCH] doc: workflow.rst: more details of current coding style process Signed-off-by: G. Paul Ziemba --- doc/developer/workflow.rst | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/doc/developer/workflow.rst b/doc/developer/workflow.rst index 80029cbd1c..972da29aa6 100644 --- a/doc/developer/workflow.rst +++ b/doc/developer/workflow.rst @@ -471,6 +471,14 @@ checkpatch.sh When convenient it is preferred that these be cleaned up inline, but this is not required. + In general, a developer should heed the information reported by checkpatch. + However, some flexibility is needed for cases where human judgement yields + better clarity than the script. Accordingly, it may be appropriate to + ignore some checkpatch.sh warnings per discussion among the submitter(s) + and reviewer(s) of a change. Misreporting of errors by the script is + possible. When this occurs, a patch for checkpatch should be added to + the pull request to correct the false error report. + If the script finds one or more WARNINGs it will exit with 1. If it finds one or more ERRORs it will exit with 2. @@ -480,6 +488,12 @@ responsibility for properly formatting your code ultimately lies on the shoulders of the submitter. As such, it is recommended to double-check the results of these tools to avoid delays in merging your submission. +In some cases, these tools modify or flag the format in ways that go beyond or +even conflict [#tool_style_conflicts]_ with the canonical documented Linux +kernel style. In these cases, the Linux kernel style takes priority; +non-canonical issues flagged by the tools are not compulsory but rather are +opportunities for discussion among the submitter(s) and reviewer(s) of a change. + **Whitespace changes in untouched parts of the code are not acceptable in patches that change actual code.** To change/fix formatting issues, please create a separate patch that only does formatting changes and @@ -875,3 +889,10 @@ particular command does. .. _GitHub: https://github.com/frrouting/frr .. _GitHub issues: https://github.com/frrouting/frr/issues + +.. rubric:: Footnotes + +.. [#tool_style_conflicts] For example, lines over 80 characters are allowed + for text strings to make it possible to search the code for them: please + see `Linux kernel style (breaking long lines and strings) `_ + and `Issue #1794 `_. -- 2.39.5