Skip to content

Move AST Evaluations to Coroutines #141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
johannes-wolf wants to merge 3 commits into
base: main
Choose a base branch
from

async: Forward For-Each Values

d60fc43
Select commit
Loading
Failed to load commit list.
Draft

Move AST Evaluations to Coroutines #141

async: Forward For-Each Values
d60fc43
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 8, 2026 in 1m 13s

Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.2% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 106 in include/simfil/expression.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 897 in src/simfil.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Complete the task associated to this "TODO" comment. 

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

Check warning on line 63 in include/simfil/model/arena.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Class has 36 methods, which is greater than the 35 authorized. Split it into smaller classes. 

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

Check warning on line 502 in src/function.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

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

Check warning on line 142 in include/simfil/function.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Pass large object "ctx" by reference to const. 

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

Check warning on line 300 in src/function.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "empty()" to check whether the container is empty or not. 

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

Check warning on line 721 in include/simfil/model/arena.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Complete the task associated to this "TODO" comment. 

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZ1uEWw8pnXB8hKEf_fQ&open=AZ1uEWw8pnXB8hKEf_fQ&pullRequest=141
View more details on SonarQubeCloud