Skip to content
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.

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

[RF] Fix pserv.C and pclient.C tutorials #13357

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

guitargeek
Copy link
Contributor

@guitargeek guitargeek commented Jul 31, 2023

Some minor fixes to make the pserv.C and pclient.C tutorials work again.

Use the same default parameters for send and recieve functions as also
used in the non-parallel sockets, such that `kDefault` doesn't need to
be passed explicitly.

Also, change return type of `TPServerSocket::Accept()` to `TPSocket*`,
which is covariant with the return typoe `TSocket*` expected by the
virtual funciton `TServerSocket::Accept()`.
The `TSocket::IsValid()` method is virtual, just like the `Send()`
method. But if the base class `TSocket::Send()` method is used in the
overriding classes (like TPSocket), then the validity check needs to be
the original one of the `TSocket`, and not dispatched virtually.
@guitargeek guitargeek self-assigned this Jul 31, 2023
@guitargeek guitargeek requested a review from gganis as a code owner July 31, 2023 14:56
@sonatype-lift
Copy link

sonatype-lift bot commented Jul 31, 2023

Sonatype Lift is retiring

Sonatype Lift will be retiring on Sep 12, 2023, with its analysis stopping on Aug 12, 2023. We understand that this news may come as a disappointment, and Sonatype is committed to helping you transition off it seamlessly. If you’d like to retain your data, please export your issues from the web console.
We are extremely grateful and thank you for your support over the years.

📖 Read about the impacts and timeline

@phsft-bot
Copy link
Collaborator

Starting build on ROOT-debian10-i386/soversion, ROOT-performance-centos8-multicore/cxx17, ROOT-ubuntu2204/nortcxxmod, ROOT-ubuntu2004/python3, mac11/noimt, mac12arm/cxx20, windows10/cxx14
How to customize builds

@guitargeek guitargeek changed the title [RF] Fix pserv.C and pclientC tutorials [RF] Fix pserv.C and pclient.C tutorials Jul 31, 2023
@github-actions
Copy link

Test Results

         9 files           9 suites   1d 19h 34m 9s ⏱️
  2 475 tests   2 471 ✔️ 0 💤 4
21 174 runs  21 170 ✔️ 0 💤 4

For more details on these failures, see this check.

Results for commit 2370101.

Copy link
Member

@Axel-Naumann Axel-Naumann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @guitargeek

We should mark TPSocket* as legacy interfaces. @couet could you submit a follow-up PR that marks this as legacy and says "please use one of the many network libraries instead, such as Boost Asio"?

@guitargeek guitargeek merged commit 8ef5d04 into root-project:master Aug 1, 2023
11 of 15 checks passed
@guitargeek guitargeek deleted the sockets branch August 1, 2023 08:36
@phsft-bot
Copy link
Collaborator

Build failed on ROOT-debian10-i386/soversion.
See console output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants