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

Fixed a lot of const stuff #26

Merged
merged 2 commits into from
Sep 27, 2023
Merged

Fixed a lot of const stuff #26

merged 2 commits into from
Sep 27, 2023

Conversation

saladuit
Copy link
Owner

Handled all of your comments
2. Made everything const and the variables and return types that could be made a reference to a reference
3. Also rerouted all warnings, errors, and fatalities to cerr.
4. Furthermore fixed some minor bugs

@saladuit saladuit added the 🔧refactor Is it easy to change? label Sep 26, 2023
@saladuit saladuit linked an issue Sep 26, 2023 that may be closed by this pull request
4 tasks
@saladuit saladuit merged commit 972f462 into main Sep 27, 2023
2 checks passed
Copy link
Collaborator

@Tentanus Tentanus left a comment

Choose a reason for hiding this comment

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

nice work. i've allready done the edits in my next pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧refactor Is it easy to change?
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

HTTPServer Class
2 participants