Skip to content

examples/threaded-ssl: delete in favor of examples/threaded #20002

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

Jump to bottom
Closed
vszakats wants to merge 8 commits into from
Closed

examples/threaded-ssl: delete in favor of examples/threaded #20002

vszakats wants to merge 8 commits into from

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Dec 16, 2025 •
edited
Loading

After applying a recent fix made to threaded.c (formerly
multithread.c) to threaded-ssl.c, syncing and updating comments,
the two examples turned out to be identical except their test URLs.

Delete one of them to avoid duplication.

Also:

  • examples/threaded: scope a variable.
  • examples/threaded: merge comments from its deleted sibling.

Follow-up to 61273f5 #20001
Follow-up to 971e8d6 #19526 #19524

https://github.com/curl/curl/pull/20002/files?w=1

  • But, is it useful to have two, almost identical multi-threaded examples? → [NO]

@vszakats vszakats changed the title examples/threaded-ssl: apply race condition fix examples/threaded-ssl: delete in favor of examples/threaded.c Dec 16, 2025
@vszakats vszakats changed the title examples/threaded-ssl: delete in favor of examples/threaded.c examples/threaded-ssl: delete in favor of examples/threaded Dec 16, 2025
@vszakats vszakats closed this in e88209d Dec 16, 2025
@vszakats vszakats deleted the exmt branch December 16, 2025 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

documentation tidy-up

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats