SDL icon indicating copy to clipboard operation
SDL copied to clipboard

Merge SDL_CreateThread

Open icculus opened this issue 3 years ago • 2 comments

Merge SDL_CreateThread and SDL_CreateThreadWithStackSize into one function, remove the hint for specifying stack size.

(And, maybe, if we add thread affinity, add that to SDL_CreateThread too, but that can come later.

icculus avatar Nov 26 '22 14:11 icculus

If we add thread affinity too, that needs to come before ABI stabilizes in 3.2.0.

slouken avatar Nov 28 '22 17:11 slouken

I'm assign this and #6636 to myself for later, but it's going to be a bit before all the SDL3 migration fires are out and I can actually write new code. :)

icculus avatar Nov 28 '22 19:11 icculus

Remind me again why we need to do this? Most people don't need to specify the stack size, so it seems reasonable to leave the simpler entry point in the Simple DirectMedia Layer API :)

slouken avatar Jan 18 '24 13:01 slouken

This was just a note about how we tacked on a second API in SDL2 to fill a need, but if it makes sense to keep a more complex thing separate, I'm fine with that.

I'm not sure thread affinity is worth adding at this time, either, honestly.

icculus avatar Jan 18 '24 14:01 icculus

Okay, let's go ahead and close this for now.

slouken avatar Jan 18 '24 14:01 slouken