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

🌍 Wrapping asio lib to create a custom network library (EPI-40) #10

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

6im0n
Copy link
Collaborator

@6im0n 6im0n commented Oct 1, 2024

This pull request introduces significant changes to the R-Type-Reborn project, focusing on adding network capabilities using the Asio library, restructuring the project, and providing documentation. Key changes include adding client-server network functionality, updating the build configuration, and including a detailed installation guide for vcpkg.

Network Functionality:

  • src/network/client/Client.cpp and src/network/client/Client.hpp: Implemented a Client class that handles connecting, sending, receiving, and disconnecting from a server using both TCP and UDP protocols. [1] [2]
  • src/network/server/Server.cpp and src/network/server/Server.hpp: Implemented a Server class that handles starting, stopping, sending, and receiving data, as well as managing client connections. [1] [2]

Build Configuration:

  • CMakeLists.txt: Updated the build configuration to include the Asio library and added subdirectories for network components.
  • src/network/CMakeLists.txt: Added CMake configuration for building the network library, including linking with Asio.

Documentation: (outdated)

Code Cleanup:

Test Files:

  • tests/client/main.cpp: Added a test client that demonstrates connecting to a server and sending a message.
  • tests/server/main.cpp: Added a test server that demonstrates starting the server and handling incoming connections.

Copy link

linear bot commented Oct 1, 2024

EPI-40 Wrap Asio

@6im0n 6im0n marked this pull request as draft October 1, 2024 17:19
@6im0n 6im0n marked this pull request as ready for review October 2, 2024 00:20
@6im0n 6im0n requested review from RenardFute, AubaneNourry, Marius-P1 and landryarki and removed request for AubaneNourry October 2, 2024 00:20
@6im0n 6im0n self-assigned this Oct 2, 2024
@6im0n 6im0n added enhancement New feature or request network feature or issue related to the network side labels Oct 2, 2024
@6im0n 6im0n changed the title Feature/epi 40 wrap asio 🌍 Wrapping asio lib to create a custom network library (EPI-40) Oct 2, 2024
Copy link
Collaborator

@landryarki landryarki left a comment

Choose a reason for hiding this comment

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

💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request network feature or issue related to the network side
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants