From 1e8f5f79e1b6ae2b4115df280c626b71c54a5bb6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Sun, 5 Feb 2023 22:04:45 +0100 Subject: [PATCH] docs/CODING_STYLE: add sentence about redirection operators --- docs/CODING_STYLE.md | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/docs/CODING_STYLE.md b/docs/CODING_STYLE.md index 516207071bb..9a355835368 100644 --- a/docs/CODING_STYLE.md +++ b/docs/CODING_STYLE.md @@ -122,7 +122,8 @@ SPDX-License-Identifier: LGPL-2.1-or-later - Empty lines to separate code blocks are a good thing, please add them abundantly. However, please stick to one at a time, i.e. multiple empty lines - immediately following each other are not OK. Also, we try to keep function calls and their immediate error handling together. Hence: + immediately following each other are not OK. Also, we try to keep function + calls and their immediate error handling together. Hence: ```c /* → empty line here is good */ @@ -131,7 +132,9 @@ SPDX-License-Identifier: LGPL-2.1-or-later if (r < 0) return log_error_errno(r, "Some function failed: %m"); /* → empty line here is good */ - ``` + +- In shell scripts, do not use whitespace after the redirection operator + (`>some/file` instead of `> some/file`, `<