Skip to content

Merge pull request #136 from Klebert-Engineering/noserde

87a0469
Select commit
Loading
Failed to load commit list.
Merged

Release v0.7.0 #133

Merge pull request #136 from Klebert-Engineering/noserde
87a0469
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 16, 2026 in 1m 15s

Annotations

Check warning on line 214 in test/token.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this escape sequence with \x{FF}.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkjt2jzZZj3JTdzz&open=AZxsfkjt2jzZZj3JTdzz&pullRequest=133

Check failure on line 702 in include/simfil/model/arena.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

"std::forward" is never called on this forwarding reference argument.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZz3C3CIh13COcKo3SVW&open=AZz3C3CIh13COcKo3SVW&pullRequest=133

Check failure on line 637 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0K&open=AZxsfkxo2jzZZj3JTd0K&pullRequest=133

Check warning on line 395 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of the constructor's initializer list for data member "model_". It is redundant with default initialization behavior.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOk_7y2qCWgr-iBnj&open=AZwpOk_7y2qCWgr-iBnj&pullRequest=133

Check warning on line 228 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Simplify this code by using "std::optional" member function "value_or".

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0A&open=AZxsfkxo2jzZZj3JTd0A&pullRequest=133

Check failure on line 616 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0H&open=AZxsfkxo2jzZZj3JTd0H&pullRequest=133

Check failure on line 623 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0I&open=AZxsfkxo2jzZZj3JTd0I&pullRequest=133

Check warning on line 99 in include/simfil/byte-array.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Convert this string literal to a raw string literal.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkph2jzZZj3JTdz3&open=AZxsfkph2jzZZj3JTdz3&pullRequest=133

Check warning on line 130 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

"std::move" is never called on this rvalue reference argument.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZzSVepBXN7jUgP8U6fU&open=AZzSVepBXN7jUgP8U6fU&pullRequest=133

Check failure on line 119 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "explicit" keyword to this constructor.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZzSVepBXN7jUgP8U6fT&open=AZzSVepBXN7jUgP8U6fT&pullRequest=133

Check failure on line 102 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the "explicit" keyword to this constructor.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOk_7y2qCWgr-iBnh&open=AZwpOk_7y2qCWgr-iBnh&pullRequest=133

Check warning on line 93 in src/diagnostics.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the init-statement to declare "allFalse" inside the if statement.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZz3C2_sh13COcKo3SVU&open=AZz3C2_sh13COcKo3SVU&pullRequest=133

Check failure on line 630 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0J&open=AZxsfkxo2jzZZj3JTd0J&pullRequest=133

Check warning on line 168 in include/simfil/model/model.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Empty class members should be marked as "[[no_unique_address]]"

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOlCny2qCWgr-iBno&open=AZwpOlCny2qCWgr-iBno&pullRequest=133

Check warning on line 13 in include/simfil/model/nodes.impl.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant cast.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZz3C3F-h13COcKo3SVZ&open=AZz3C3F-h13COcKo3SVZ&pullRequest=133

Check failure on line 61 in include/simfil/model/model.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Member variables should not be "protected".

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOlCny2qCWgr-iBnn&open=AZwpOlCny2qCWgr-iBnn&pullRequest=133

Check failure on line 553 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0C&open=AZxsfkxo2jzZZj3JTd0C&pullRequest=133

Check warning on line 102 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not use the constructor's initializer list for data member "data_". Use the in-class initializer instead.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOk_7y2qCWgr-iBnl&open=AZwpOk_7y2qCWgr-iBnl&pullRequest=133

Check failure on line 574 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use has_value() or another member function to clarify that the code tests the presence of a value in the "optional", not the contained "long" value itself.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0F&open=AZxsfkxo2jzZZj3JTd0F&pullRequest=133

Check warning on line 505 in include/simfil/model/nodes.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant access specifier; it does not change the accessibility level.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZwpOk_7y2qCWgr-iBnk&open=AZwpOk_7y2qCWgr-iBnk&pullRequest=133

Check failure on line 47 in include/simfil/model/nodes.impl.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Explicitly capture all local variables required in this lambda.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkvM2jzZZj3JTdz_&open=AZxsfkvM2jzZZj3JTdz_&pullRequest=133

Check warning on line 611 in include/simfil/operator.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace with the version of "std::ranges::lexicographical_compare" that takes a range.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkxo2jzZZj3JTd0G&open=AZxsfkxo2jzZZj3JTd0G&pullRequest=133

Check warning on line 71 in include/simfil/byte-array.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "std::memcpy" invocation with "std::bit_cast".

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkph2jzZZj3JTdz2&open=AZxsfkph2jzZZj3JTdz2&pullRequest=133

Check warning on line 75 in include/simfil/model/nodes.impl.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant cast.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZz3C3F-h13COcKo3SVa&open=AZz3C3F-h13COcKo3SVa&pullRequest=133

Check warning on line 46 in src/model/json.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The result of "std::move" should not be passed as a const reference.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZxsfkol2jzZZj3JTdz1&open=AZxsfkol2jzZZj3JTdz1&pullRequest=133