savushkin-r-d / ptusa_main

Programmable automation controller (PAC) main program
MIT License
4 stars 20 forks source link

Fix sockets processing #748

Closed idzm closed 2 weeks ago

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.45%. Comparing base (0f9c13b) to head (12cbd34). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #748 +/- ## ======================================= Coverage 44.44% 44.45% ======================================= Files 93 93 Lines 27578 27577 -1 Branches 2883 2883 ======================================= + Hits 12257 12259 +2 + Misses 15316 15313 -3 Partials 5 5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

idzm commented 2 weeks ago

Так как в процессе обработки подключений могут не только добавляться новые сокеты, но и удаляться существующие, необходимо использовать sst.size():


for ( u_int i = 0; i < sst.size(); i++ ) /* scan all possible sockets */
sonarcloud[bot] commented 2 weeks ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud