Skip to content

Add Windows client/server socket examples and update Makefiles - #8

Open
niki-mardari wants to merge 1 commit into
mainfrom
slave
Open

Add Windows client/server socket examples and update Makefiles#8
niki-mardari wants to merge 1 commit into
mainfrom
slave

Conversation

@niki-mardari

Copy link
Copy Markdown
Collaborator

Made platform windows server example, creates a windows TCP socket using Winsock.h, binds it to port 1500, listens for a client, sends a simple defined message to the client after connection establishment.

image

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The -j flag enables the use of parallel building which is useful especially when the project gets big.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Nice work I will have a go at running this code but if it worked for you I will trust you - "Source trust me bro 馃槅"

@niki-mardari niki-mardari Apr 8, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

That's a good precaution. I would still test it on your machine to be sure. I should have tested the code more, if it didn't work on your machine. I didn't think of it, ran it in powershell got excited that it finally worked and thought that's it...

I'll look into testing code with Github actions before pull requesting. I should have put a Readme to explain what environment was used, a requirements and how to build/ run...

"Trust me Bro" is not something I want to do.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I will add *.txt to the .gitignore file. Also best practice is to create an output directory and put the files there.

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.

2 participants