{"thread":{"id":"62295","subject":"[PATCH 1/3] clang-format: don't enforce the column limit","startedAt":"2024-10-09T12:51:14Z","lastAt":"2024-10-09T12:58:44Z","messageCount":2,"participants":["Karthik Nayak"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"504551","messageId":"CAOLa=ZQrCxU_tUcFsrAjTxq2ufCNk=zox-fsa_KMenyCJr5gUg@mail.gmail.com","threadId":"62295","inReplyTo":"20241009125109.24422-1-karthik.188@gmail.com","subject":"[PATCH 1/3] clang-format: don't enforce the column limit","fromName":"Karthik Nayak","fromEmail":"karthik.188@gmail.com","sentAt":"2024-10-09T12:51:13Z","receivedAt":"2024-10-09T12:51:14Z","isPatch":true,"sender":{"key":"karthik.188@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1786334?v=4"},"body":"The current value for the column limit is set to 80. While this is as\nexpected, we often prefer readability over this strict limit. This means\nit is common to find code which extends over 80 characters. So let's\nchange the column limit to be 0 instead. This ensures that the formatter\ndoesn't complain about code strictly not following the column limit.\n\nSigned-off-by: Karthik Nayak <karthik.188@gmail.com>\n---\n .clang-format | 6 +++++-\n 1 file changed, 5 insertions(+), 1 deletion(-)\n\ndiff --git a/.clang-format b/.clang-format\nindex 41969eca4b..38910a3a53 100644\n--- a/.clang-format\n+++ b/.clang-format\n@@ -12,7 +12,11 @@ UseTab: Always\n TabWidth: 8\n IndentWidth: 8\n ContinuationIndentWidth: 8\n-ColumnLimit: 80\n+\n+# While we recommend keeping column limit to 80, we don't want to\n+# enforce it as we generally are more lenient with this rule and\n+# prefer to prioritize readability.\n+ColumnLimit: 0\n\n # C Language specifics\n Language: Cpp\n-- \n2.46.2\n"},{"id":"504555","messageId":"CAOLa=ZTY+XYQE8GRz9qigDqiu=oJUhSqEGhXevkuW_tqaoHyxA@mail.gmail.com","threadId":"62295","inReplyTo":"CAOLa=ZQrCxU_tUcFsrAjTxq2ufCNk=zox-fsa_KMenyCJr5gUg@mail.gmail.com","subject":"Re: [PATCH 1/3] clang-format: don't enforce the column limit","fromName":"Karthik Nayak","fromEmail":"karthik.188@gmail.com","sentAt":"2024-10-09T12:58:43Z","receivedAt":"2024-10-09T12:58:44Z","isPatch":true,"sender":{"key":"karthik.188@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1786334?v=4"},"body":"Karthik Nayak <karthik.188@gmail.com> writes:\n\n> The current value for the column limit is set to 80. While this is as\n> expected, we often prefer readability over this strict limit. This means\n> it is common to find code which extends over 80 characters. So let's\n> change the column limit to be 0 instead. This ensures that the formatter\n> doesn't complain about code strictly not following the column limit.\n>\n\nSeems like this mail got double posted, please ignore. The original\nthread [1] has the full set of patches.\n\n[1]: https://lore.kernel.org/git/CAOLa=ZRvFBhageS65uE5enzLBz7H_CAvvnEcPsi_QAi0exRx2w@mail.gmail.com/#r\n"}]}