Skip to content

cppcheck: add rationale to suppressed warnings #1895

Closed
#10010
Closed
cppcheck: add rationale to suppressed warnings#1895
#10010
Labels
Area: docArea: DocumentationCommunity: good first issueThis issue is good for newcomers to RIOT to be addressed / implementedType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
@N8Fear

Description

@N8Fear
N8Fear
opened on Oct 28, 2014

All suppressions for warnings/errors in cppcheck should be annotated with a rationale why the warning/error is suppressed instead of being fixed.

Also it would make sense to add a statement about this to the coding conventions in the wiki, if there is consent about this.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Area: docArea: DocumentationCommunity: good first issueThis issue is good for newcomers to RIOT to be addressed / implementedType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions