Skip to content

Review of pthread_* return values - #793

Open
mlechon wants to merge 121 commits into
EpicGames:devfrom
mlechon:phtread_return_values
Open

mlechon wants to merge 121 commits into
EpicGames:devfrom
mlechon:phtread_return_values

Conversation

@mlechon

@mlechon mlechon commented May 23, 2026

Copy link
Copy Markdown

(Hi and thanks for the linux port! Feel free to close/disregard this PR without comment.)

There are some checks of pthread_* return values that don't look right, if we go by the manual pages. In general, these functions return 0 when successful and an error code (not necessarily -1) on failure. Also, pthread_mutex_init always returns 0, so there's no point in checking it.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants