Skip to content

Refactored shader loader and updated rendering pipeline. Formatted the code for better readability, made the DemoShaderLoader implementation safer than before, and rewrote the code using modern C++ practices to ensure every line is easy to understand. - #7

Open
Dhaval572 wants to merge 1 commit into
meemknight:masterfrom
Dhaval572:master

Conversation

@Dhaval572

Copy link
Copy Markdown

Refactored demoShaderLoader for improved naming consistency and modern C++ practices. Replaced legacy OpenGL pipeline in main.cpp with VAO/VBO and programmable shaders, updating to OpenGL 4.6 core profile. Added new Sample.vert and Sample.frag shaders, removed old vertex.vert and fragment.frag. Updated resource handling in CMakeLists.txt for relative paths. Minor improvements to openglDebug.cpp formatting.

Refactored demoShaderLoader for improved naming consistency and modern C++ practices. Replaced legacy OpenGL pipeline in main.cpp with VAO/VBO and programmable shaders, updating to OpenGL 4.6 core profile. Added new Sample.vert and Sample.frag shaders, removed old vertex.vert and fragment.frag. Updated resource handling in CMakeLists.txt for relative paths. Minor improvements to openglDebug.cpp formatting.
@Dhaval572 Dhaval572 changed the title Refactor shader loader and update rendering pipeline Refactor shader loader and update rendering pipeline and formatted the code in very readable way and i removed the made the DemoShader Loader implementation code safer than old and made every line of code easy to understand by modern c++ Sep 9, 2025
@Dhaval572 Dhaval572 changed the title Refactor shader loader and update rendering pipeline and formatted the code in very readable way and i removed the made the DemoShader Loader implementation code safer than old and made every line of code easy to understand by modern c++ Refactored shader loader and updated rendering pipeline. Formatted the code for better readability, made the DemoShaderLoader implementation safer than before, and rewrote the code using modern C++ practices to ensure every line is easy to understand. Sep 9, 2025
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.

1 participant