docs: add C-API snippets alongside C++ RAII snippets#1
Open
walcht wants to merge 6 commits intogpx1000:use-multi-language-in-sourcefrom
Open
docs: add C-API snippets alongside C++ RAII snippets#1walcht wants to merge 6 commits intogpx1000:use-multi-language-in-sourcefrom
walcht wants to merge 6 commits intogpx1000:use-multi-language-in-sourcefrom
Conversation
- add equivalent C-API snippets - add a small introduction to the views concept in Vulkan - adjust for consistent use of designated initializers - add consistent 80-column width formatting - improve wordings and structure
- add equivalent C-API snippets for the 01_Swap_chain chapter - remove redundant code snippets - add consistent formatting (same clang-format applied to attachments/ except with a 100 column limit) - fix inconsistencies in C++ RAII code snippets (e.g., using non-declared variables, declaring imageCount then not using it, etc.) - re-adjust some paragraphs so they fit within 80 column limit (this was obviously done before but inconsistently)
- add equivalent C-API snippets for the 01_Shader_modules chapter - fix Slang syntax highlighting
- add equivalent C-API snippets for the 02_Fixed_functions chapter - readjust dynamic state section position (no reason to jump directly into it at the beginning of the chapter) - format code snippets - misc restructuring (dynamic state is all over the place, optional fields are omitted inconsistently, etc.)
- add equivalent C-API snippets for the 03_Render_passes chapter - fix snippets formatting (using attachments/.clang-format with 100 column limit) - readjust text for consistent 80 column width (same as initial chapters)
- add equivalent C-API snippets for the 04_Conclusion chapter - format code snippets - restructure code (no reason to refill the VkGraphicsPipelineCreateInfo struct again - it was already filled in previous chapter) - misc restructuring
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add C-API and improve/address misc issues:
See: this closed PR