Skip to content

Commit 2fc14d3

Browse files
committed
DpCatalog: move constructor body off the signature line
Addresses CodeQL "More than one statement per line" alert introduced by the previous commit, which collapsed the constructor signature and the empty body onto the same line. Adds an explanatory comment inside the body to keep it on its own line; clang-format would otherwise re-collapse the empty `{}` under the repo's Chromium-based style. Behavior unchanged.
1 parent ec30722 commit 2fc14d3

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

Svc/DpCatalog/DpCatalog.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ static_assert(DP_MAX_FILES > 0, "Configuration DP_MAX_FILES must be positive");
2121
// Component construction and destruction
2222
// ----------------------------------------------------------------------
2323

24-
DpCatalog ::DpCatalog(const char* const compName) : DpCatalogComponentBase(compName) {}
24+
DpCatalog ::DpCatalog(const char* const compName) : DpCatalogComponentBase(compName) {
25+
// Members are default-initialized via in-class initializers in the header.
26+
}
2527

2628
void DpCatalog::configure(Fw::FileNameString directories[DP_MAX_DIRECTORIES],
2729
FwSizeType numDirs,

0 commit comments

Comments
 (0)