Skip to content

Maintenance: Add missing stub_fde.cc#2359

Open
yadij wants to merge 3 commits intosquid-cache:masterfrom
yadij:arc-stub_fde-1
Open

Maintenance: Add missing stub_fde.cc#2359
yadij wants to merge 3 commits intosquid-cache:masterfrom
yadij:arc-stub_fde-1

Conversation

@yadij
Copy link
Contributor

@yadij yadij commented Jan 19, 2026

Also moved fdNFree() declaration to fd.h (to match its definition
location) and polished its stub's return value.

@rousskov rousskov self-requested a review January 19, 2026 21:52
Copy link
Contributor

@rousskov rousskov left a comment

Choose a reason for hiding this comment

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

I added PR description to discuss fdNFree() changes.

@rousskov rousskov added S-waiting-for-author author action is expected (and usually required) S-could-use-an-approval An approval may speed this PR merger (but is not required) labels Jan 27, 2026
squid-anubis pushed a commit that referenced this pull request Jan 27, 2026
Also moved `fdNFree()` declaration to `fd.h` (to match its definition
location) and polished its stub's return value.
@squid-anubis squid-anubis added M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-failed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels and removed M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 27, 2026
@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels and removed S-waiting-for-author author action is expected (and usually required) labels Jan 28, 2026
@squid-anubis squid-anubis added the M-abandoned-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels label Jan 30, 2026
@yadij yadij removed M-failed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-abandoned-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 31, 2026
squid-anubis pushed a commit that referenced this pull request Jan 31, 2026
Also moved `fdNFree()` declaration to `fd.h` (to match its definition
location) and polished its stub's return value.
@squid-anubis squid-anubis added M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-failed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-abandoned-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels and removed M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 31, 2026
@kinkie kinkie removed M-failed-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels M-abandoned-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Feb 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants