Skip to content

Issue 2769#39

Open
thomas-bc wants to merge 24 commits intoLeStarch:develfrom
thomas-bc:issue-2769
Open

Issue 2769#39
thomas-bc wants to merge 24 commits intoLeStarch:develfrom
thomas-bc:issue-2769

Conversation

@thomas-bc
Copy link
Copy Markdown
Collaborator

Related Issue(s)
Has Unit Tests (y/n)
Documentation Included (y/n)
Generative AI was used in this contribution (y/n)

Change Description

Rationale

Testing/Review Recommendations

Future Work

AI Usage (see policy)

//!
//! @param value The value of the FSS object.
//!
FileSizeSensitive(U64 value);

Check warning

Code scanning / CppCheck

Single-parameter constructors should be marked explicit. Warning

Single-parameter constructors should be marked explicit.
void verifyObject(FilePacket::Finished& finished);
};

/*

Check warning

Code scanning / CppCheck

Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. Warning test

Complex multi-line /*...*/-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments.
};

/*
* Test Metadata helpers.

Check warning

Code scanning / CppCheck

Could not find end of multi-line comment Warning test

Could not find end of multi-line comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants