Skip to content

docs: few minor fixes. #4653

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

Merged
merged 4 commits into from
Jul 25, 2023
Merged

docs: few minor fixes. #4653

merged 4 commits into from
Jul 25, 2023

Conversation

spacemiqote
Copy link
Contributor

@spacemiqote spacemiqote commented Jul 25, 2023

Hi! I hope you are having a great day.
I am submitting a pull request with the following changes:

  1. Updated explanation to align with code and fix a minor grammar mistake in the documentation related to C Bit Fields. 1cfca66
  2. Clarify the description regarding the default size in the documentation related to Storage and Alignment of Structures. 532242a
  3. Fix a comment formatting error in the documentation related to Union Declarations. f92be36

Thank you for your time and consideration in reviewing these changes.
Best regards,
melody

Updated explanation to align with code and fix a minor grammar mistake in the documentation related to C Bit Fields.
Clarify the description regarding the default size in the documentation related to Storage and Alignment of Structures.
Fix a comment formatting error in the documentation related to Union Declarations.
@prmerger-automator
Copy link
Contributor

@moonlit-melody : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f92be36:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/c-bit-fields.md ✅Succeeded
docs/c-language/storage-and-alignment-of-structures.md ✅Succeeded
docs/c-language/union-declarations.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thanks, @moonlit-melody! I think we should tweak the description of how many bits can be in the bit field.

Updated the explanation to align with the code, taking Tyler's input into account. Thank you, Tyler!
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 1cfca66:

✅ Validation status: passed

File Status Preview URL Details
docs/c-language/c-bit-fields.md ✅Succeeded
docs/c-language/storage-and-alignment-of-structures.md ✅Succeeded
docs/c-language/union-declarations.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

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

Thanks, @moonlit-melody!

@TylerMSFT
Copy link
Collaborator

#sign-off

@Jak-MS Jak-MS merged commit 4a97921 into MicrosoftDocs:main Jul 25, 2023
@spacemiqote spacemiqote deleted the patch-1 branch July 25, 2023 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants