Skip to content

Sam modified #1336

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

Open
wants to merge 52 commits into
base: main
Choose a base branch
from

Update port.c

530cdce
Select commit
Loading
Failed to load commit list.
Open

Sam modified #1336

Update port.c
530cdce
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Nov 24, 2025 in 18s

Quality Gate failed

Failed conditions
1 Security Hotspot
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 1391 in include/task.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the misleading "//" characters. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEagjgsQoqnsxwXk&open=AZq3nEagjgsQoqnsxwXk&pullRequest=1336

Check warning on line 1444 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declare the variable "x" inside the loop. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYb&open=AZq3nEiOjgsQoqnsxwYb&pullRequest=1336

Check warning on line 3042 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYn&open=AZq3nEiOjgsQoqnsxwYn&pullRequest=1336

Check warning on line 2982 in include/task.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the misleading "//" characters. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEagjgsQoqnsxwXs&open=AZq3nEagjgsQoqnsxwXs&pullRequest=1336

Check warning on line 128 in stream_buffer.c

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=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEdnjgsQoqnsxwX7&open=AZq3nEdnjgsQoqnsxwX7&pullRequest=1336

Check warning on line 743 in include/timers.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the misleading "//" characters. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEXSjgsQoqnsxwXX&open=AZq3nEXSjgsQoqnsxwXX&pullRequest=1336

Check failure on line 873 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYV&open=AZq3nEiOjgsQoqnsxwYV&pullRequest=1336

Check warning on line 859 in stream_buffer.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEdnjgsQoqnsxwX3&open=AZq3nEdnjgsQoqnsxwX3&pullRequest=1336

Check warning on line 910 in tasks.c

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=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYY&open=AZq3nEiOjgsQoqnsxwYY&pullRequest=1336

Check failure on line 822 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 26 to the 25 allowed. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYU&open=AZq3nEiOjgsQoqnsxwYU&pullRequest=1336

Check warning on line 4465 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move this function declaration outside of this block scope, or if the intent was to declare a variable, use a syntax that avoids the most vexing parse. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYu&open=AZq3nEiOjgsQoqnsxwYu&pullRequest=1336

Check failure on line 1686 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this code to not nest more than 3 if|for|do|while|switch statements. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYf&open=AZq3nEiOjgsQoqnsxwYf&pullRequest=1336

Check warning on line 563 in include/task.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the misleading "//" characters. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEagjgsQoqnsxwXd&open=AZq3nEagjgsQoqnsxwXd&pullRequest=1336

Check warning on line 1135 in stream_buffer.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEdnjgsQoqnsxwX5&open=AZq3nEdnjgsQoqnsxwX5&pullRequest=1336

Check warning on line 718 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Make the type of this parameter a pointer-to-const. The current type of "pxTCB" is "struct tskTaskControlBlock *". 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYT&open=AZq3nEiOjgsQoqnsxwYT&pullRequest=1336

Check warning on line 3637 in tasks.c

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=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYL&open=AZq3nEiOjgsQoqnsxwYL&pullRequest=1336

Check warning on line 941 in include/FreeRTOS.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define the type with "typedef" instead of using a macro. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nETUjgsQoqnsxwXW&open=AZq3nETUjgsQoqnsxwXW&pullRequest=1336

Check warning on line 4123 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYs&open=AZq3nEiOjgsQoqnsxwYs&pullRequest=1336

Check warning on line 730 in stream_buffer.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEdnjgsQoqnsxwX1&open=AZq3nEdnjgsQoqnsxwX1&pullRequest=1336

Check warning on line 5954 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYw&open=AZq3nEiOjgsQoqnsxwYw&pullRequest=1336

Check warning on line 537 in event_groups.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define each identifier in a dedicated statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEgwjgsQoqnsxwYK&open=AZq3nEgwjgsQoqnsxwYK&pullRequest=1336

Check warning on line 1159 in include/queue.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the misleading "//" characters. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEcTjgsQoqnsxwXt&open=AZq3nEcTjgsQoqnsxwXt&pullRequest=1336

Check warning on line 3618 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Modify the macro definition so that it needs to be followed by a semicolon, or remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYp&open=AZq3nEiOjgsQoqnsxwYp&pullRequest=1336

Check warning on line 4275 in tasks.c

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=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwYN&open=AZq3nEiOjgsQoqnsxwYN&pullRequest=1336

Check warning on line 6186 in tasks.c

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these redundant parentheses. 

See more on https://sonarcloud.io/project/issues?id=FreeRTOS_FreeRTOS-Kernel&issues=AZq3nEiOjgsQoqnsxwY2&open=AZq3nEiOjgsQoqnsxwY2&pullRequest=1336
View more details on SonarQubeCloud