pmmp / ext-pmmpthread

Fork of https://github.com/krakjoe/pthreads with a revamped API and PHP 8.1+ support
Other
81 stars 17 forks source link

Rename extension #112

Closed dktapps closed 1 year ago

dktapps commented 1 year ago

As of 5.0.0, this extension now has an API that differs quite a bit from pthreads. It's also more performant and is easier to understand.

Since we don't actually have a claim to the name pthreads (this is an unofficial fork), we can't publish the extension to PECL under the pthreads name, and calling it pthreads is possibly misleading. We also can't submit official stubs to jetbrains/phpstorm-stubs, requiring users to install custom stubs for IDE development.

I plan to release 6.0.0 soon, with some class renames and namespacing to distinguish it from pthreads.

Some possible new names (with the help of PM #internals-dev and ChatGPT):

SOF3 commented 1 year ago

pmmpthreads sounds fine to me. A bit long, but it's not namespaced anyway so users don't need to type it anywhere other than during installation or dependency declaration, and internally we just call it "threading" either way

dktapps commented 1 year ago

For consistency with our other projects, I chose a namespace of pmmp\thread. pmmpthread without the s works with this too, I think.

For some reason I instinctively pmmpthreads as "pmm pthreads", while pmmpthread reads as "pmmp thread".