CI

Payload Builder supporting google.protobuf.Any #912

Sign in to view logs

Triggered via pull request March 3, 2025 19:47

lukas-he

synchronize #306

lukas-he:feature/141-builder-any-support

Status Failure

Total duration 5m 6s

Artifacts 8

ci.yml

on: pull_request

Build up-cpp and dependencies

Run up-cpp tests

Run Valgrind Memcheck

Run Valgrind ThreadCheck

Run Valgrind Helgrind

Run Valgrind DHAT

Lint C++ sources

CI status checks

Annotations

3 errors and 10 warnings

src/datamodel/builder/Payload.cpp:52:6 [clang-diagnostic-pessimizing-move]: src/datamodel/builder/Payload.cpp#L52

moving a temporary object prevents copy elision

test/coverage/datamodel/PayloadBuilderTest.cpp:133:15 [clang-diagnostic-unused-result]: test/coverage/datamodel/PayloadBuilderTest.cpp#L133

ignoring return value of function declared with 'nodiscard' attribute

Lint C++ sources

Process completed with exit code 1.

include/up-cpp/datamodel/builder/Payload.h:83:21 [misc-unused-parameters]: include/up-cpp/datamodel/builder/Payload.h#L83

parameter 's' is unused

include/up-cpp/datamodel/builder/Payload.h:84:8 [readability-identifier-naming]: include/up-cpp/datamodel/builder/Payload.h#L84

invalid case style for variable 'serializedData'

include/up-cpp/datamodel/builder/Payload.h:99:10 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L99

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

include/up-cpp/datamodel/builder/Payload.h:110:36 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L110

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

include/up-cpp/datamodel/builder/Payload.h:123:31 [readability-avoid-const-params-in-decls]: include/up-cpp/datamodel/builder/Payload.h#L123

parameter 'format' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions

src/datamodel/builder/Payload.cpp:30:29 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L30

std::move of the const variable 'value' has no effect; remove std::move() or make the variable non-const

src/datamodel/builder/Payload.cpp:58:51 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L58

std::move of the expression of the trivially-copyable type 'bool' has no effect; remove std::move()

src/datamodel/builder/Payload.cpp:61:10 [modernize-use-equals-default]: src/datamodel/builder/Payload.cpp#L61

use '= default' to define a trivial copy constructor

src/datamodel/builder/Payload.cpp:67:11 [performance-move-const-arg]: src/datamodel/builder/Payload.cpp#L67

std::move of the expression of the trivially-copyable type 'bool' has no effect; remove std::move()

src/datamodel/builder/Payload.cpp:72:19 [cert-oop54-cpp]: src/datamodel/builder/Payload.cpp#L72

operator=() does not handle self-assignment properly

Artifacts

Produced during runtime

Name	Size
build-artifacts	39.4 MB
compile-commands	1.81 KB
conan-cache	2.49 MB
test-results	125 KB
valgrind-dhat-log	2.91 KB
valgrind-helgrind-log	45.8 KB
valgrind-memcheck-log	92.1 KB
valgrind-threadcheck-log	258 KB