Skip to content

Conversation

@opbld16
Copy link
Contributor

@opbld16 opbld16 commented Sep 16, 2022

The pull request is created from main637989455525414102sync_temp to main to fix git push error for protected CLA branch

Taojunshen and others added 2 commits September 17, 2022 01:17
* Updated C6101

Mirror of my public PR. Added an example. Added proper spacing between headers and code blocks, matched formatting to my other PRs

* Updated C6101

Minor changes in response to Acrolinx report

* Updated C6101

More Acrolinx changes, split a run-on sentence

* Updated C26495

Matched formatting to my other PRs. Added more information to the example.

* Updated C26495

Fixed spelling error

* Updated C6101

Updated to the latest format

* Updated C6101

Single backticks and InOut->Inout

* Updated C26495

Updated to the new format
NOTE: This warning has no code analysis name. Per @MichaelSquires instructions, I omitted the usual keywords and 'Code analysis name:' section to match

* Update compiler-warning-level-4-c4464.md

@corob-msft 
proposal.  I don't warrant it shouldn't be worded in a better way.

* Update compiler-warning-level-4-c4464.md

typo fix

* fix: delete unnecessary asterisk

followup fix of #4136

* optimize prime test

* Update floating-point-support.md

Explain what "ulp" stands for

* Updates for cpp-docs 4161

* Update integritycheck-require-signature-check.md

* alt-text

* Update abstract-cpp-component-extensions.md

* Add syntax highlighting to atl-mfc-shared

* Updated C26495

Added code analysis ID

* alt-text updates

* customer fix

* minor updates

* Incorporate changes in rewrite

Updates for style and clarity. Also give it an Acrolinx pass.

* Clarify version support

* Tweak language for minimum support

* Add code style, comment

@MugBergerFries 
It's a good idea to use code styling for mentions of variable or class names (especially when they're easily confused generic names like "value").
I also added a comment to the fixed code sample to amplify the change and what it was doing.

* acrolinx

* cleanup pass

Co-authored-by: Samuel Berger <sjamesberger@gmail.com>
Co-authored-by: Brad Litterell <49081806+bradlitterell@users.noreply.github.com>
Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Colin Robertson <3836425+corob-msft@users.noreply.github.com>
Co-authored-by: opbld16 <opbld16@microsoft.com>
Co-authored-by: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com>
Co-authored-by: Courtney Wales <62625502+Court72@users.noreply.github.com>
Co-authored-by: opbld15 <opbld15@microsoft.com>
Co-authored-by: TylerMSFT <Tyler.Whitney@microsoft.com>
Co-authored-by: Bryan Gold <101299717+19BMG00@users.noreply.github.com>
Co-authored-by: Edward Breeveld <42862235+EddieBreeveld@users.noreply.github.com>
Co-authored-by: Jeff Borsecnik <36546697+jborsecnik@users.noreply.github.com>
Co-authored-by: jsuther1974 <jsuther@microsoft.com>
Co-authored-by: Austin Morton <apmorton@users.noreply.github.com>
Co-authored-by: Jak Koke <v-jokoke@microsoft.com>
Co-authored-by: HO-COOH <42881734+HO-COOH@users.noreply.github.com>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: James Barnett <v-jabarnett@microsoft.com>
Co-authored-by: Dennis Rea <v-denrea@microsoft.com>
@prmerger-automator
Copy link
Contributor

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

@colin-home colin-home merged commit efc1a1e into main Sep 16, 2022
@colin-home colin-home deleted the main637989455525414102sync_temp branch September 16, 2022 21:03
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.

5 participants