Skip to content

Conversation

@Mateusz1223
Copy link

@Mateusz1223 Mateusz1223 commented Sep 24, 2025

Description

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (refactoring, style fixes, git/CI config, submodule management, no code logic changes)

How Has This Been Tested?

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing linter checks and tests passed.
  • My changes generate no new compilation warnings for any of the targets.

Special treatment

shutdown(common.rec.listeners[i].fd, SHUT_RDWR);

while ((clientfd = accept(common.rec.listeners[i].fd, NULL, NULL)) != -1) {
while (read(clientfd, buffer, sizeof(buffer)) > 0) {}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[clang-format-pr] reported by reviewdog 🐶
suggested fix

Suggested change
while (read(clientfd, buffer, sizeof(buffer)) > 0) {}
while (read(clientfd, buffer, sizeof(buffer)) > 0) { }

freePort = false;

/*
If this condition is not true it means that the port was aquired by other thread, most

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[codespell] reported by reviewdog 🐶
aquired ==> acquired

@Mateusz1223 Mateusz1223 force-pushed the Mateusz1223/host_msg branch 3 times, most recently from d42e42a to 9a237fb Compare September 24, 2025 16:20
@github-actions
Copy link

github-actions bot commented Sep 24, 2025

Unit Test Results

9 277 tests  +1 040   8 688 ✅ +975   55m 26s ⏱️ + 14m 32s
  545 suites +   66     589 💤 + 65 
    1 files   ±    0       0 ❌ ±  0 

Results for commit 3de746b. ± Comparison against base commit b5cf78f.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants