poco icon indicating copy to clipboard operation
poco copied to clipboard

Socket::select() should not be deprecated

Open obiltschnig opened this issue 9 months ago • 4 comments

I don't think Socket::select() should be deprecated. It's still useful for some use cases where one just needs to check a few sockets.

obiltschnig avatar Feb 27 '25 18:02 obiltschnig

Even we are using it and this change forces us to do a lot more code changes. Though, we can compile POCO using POCO_SILENCE_DEPRECATED to suppress this warning. But this will be our last resort as this not only suppresses for Socket::select, but for others APIs too.

FYI - currently, we are using older version e.g. 1.11.x and migrating to 1.14.x.

SinghRajenM avatar Mar 03 '25 03:03 SinghRajenM

select() was marked as deprecated in #1459.

It seems like a good idea to deprecate select().

matejk avatar Mar 19 '25 11:03 matejk

it would make sense to keep it if we make it just a simple wrapper for the platform select(). Whoever needs poll, should use Socket::poll().

aleks-f avatar Mar 19 '25 14:03 aleks-f

That would make sense. What I'd like to avoid is forcing people (including myself) to rewrite their code to use PollSet instead of select() for use cases where select() is just fine.

obiltschnig avatar Mar 19 '25 14:03 obiltschnig