To ensure a uniform code style, I suggest adding a .clang-fo

Sounds good, only strong preferences I have are: 4 space inden

I suggest this configuration: <a href="https://github.com/RUrlus/carma/files/48867

Only thing I don't like about the config is: <div class="snippet-clipboard-content

Thanks for the work, closed by <a class="issue-link js-issue-link" data-error-text="Fa

Add formatting guide (aka clang-format config) about carma HOT 6 CLOSED

rurlus commented on August 17, 2024

Add formatting guide (aka clang-format config)

from carma.

Comments (6)

RUrlus commented on August 17, 2024

Sounds good, only strong preferences I have are:

4 space indent
opening on same line, close on new line, e.g.

if (something) {
    do_something_here
}

Indent within namespaces I am fine with either indent or no indent

from carma.

hpwxf commented on August 17, 2024

Ok; understood 👍

from carma.

hpwxf commented on August 17, 2024

I suggest this configuration:
clang-format.txt

You can try it here https://zed0.co.uk/clang-format-configurator/ using upload feature.

from carma.

RUrlus commented on August 17, 2024

Only thing I don't like about the config is:

std::vector<uint32_t> returnVector( uint32_t* LongNameForParameter1,
                                    double* LongNameForParameter2,
                                    const float& LongNameForParameter3,
                                    const std::map<std::string,int32_t>& LongNameForParameter4) {

Preferring:

std::vector<uint32_t> returnVector(
    uint32_t* LongNameForParameter1,
    double* LongNameForParameter2,
    const float& LongNameForParameter3,
    const std::map<std::string, int32_t>& LongNameForParameter4
) {

However this doesn't seem possible, the closest I got:

std::vector<uint32_t> returnVector(
    uint32_t* LongNameForParameter1,
    double* LongNameForParameter2,
    const float& LongNameForParameter3,
    const std::map<std::string, int32_t>& LongNameForParameter4) {

What are your thoughts?

config:

---
BasedOnStyle: Chromium
AllowShortBlocksOnASingleLine: 'false'
AllowShortCaseLabelsOnASingleLine: 'false'
AllowShortIfStatementsOnASingleLine: 'false'
AllowShortLoopsOnASingleLine: 'false'
AlwaysBreakTemplateDeclarations: 'Yes'
BinPackArguments: 'false'
BinPackParameters: 'false'
BreakBeforeBinaryOperators: All
BreakInheritanceList: BeforeComma
ColumnLimit: '120'
CompactNamespaces: 'false'
FixNamespaceComments: 'true'
IndentWidth: 4
UseTab: Never
ObjCBlockIndentWidth: 4
ContinuationIndentWidth: 4
AlignAfterOpenBracket: AlwaysBreak
IndentCaseLabels: true
AllowAllParametersOfDeclarationOnNextLine: false

from carma.

hpwxf commented on August 17, 2024

Except ObjCBlockIndentWidth which is for Objective-C code, I've updated clang-format following your config (cf #37).
(and I don't know how to do a style closer to your preferred style).

from carma.

RUrlus commented on August 17, 2024

Thanks for the work, closed by #37

from carma.

Recommend Projects