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

Fix typo in code comment #3109

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

lkb85
Copy link

@lkb85 lkb85 commented Feb 5, 2025

[Fix typos in code comments]

Motivation:

To correct minor typographical errors in code comments for better readability and understanding.

Modifications:

_Fixed typos in comments within the following files:

  • Sources/CNIOAtomics/src/cpp_magic.h
  • Sources/CNIOLLHTTP/include/c_nio_llhttp.h_

Result:

Improved clarity and accuracy of code comments. No functional changes to the code.

N/A

@@ -219,7 +219,7 @@
*
* 1. The preprocessor expands the arguments to _IF casting the condition to '0'
* or '1'.
* 2. The casted condition is concatencated with _IF_ giving _IF_0 or _IF_1.
* 2. The casted condition is concatenated with _IF giving _IF_0 or _IF_1.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* 2. The casted condition is concatenated with _IF giving _IF_0 or _IF_1.
* 2. The casted condition is concatenated with _IF_ giving _IF_0 or _IF_1.

@@ -675,7 +675,7 @@ llhttp_errno_t c_nio_llhttp_execute(llhttp_t* parser, const char* data, size_t l
* Requests without `Content-Length` and other messages might require treating
* all incoming bytes as the part of the body, up to the last byte of the
* connection. This method will invoke `on_message_complete()` callback if the
* request was terminated safely. Otherwise a error code would be returned.
* request was terminated safely. Otherwise an error code would be returned.
Copy link
Contributor

Choose a reason for hiding this comment

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

We can't change this file: it's vendored from llhttp.

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.

3 participants