Skip to content

Conversation

@bobis33
Copy link
Owner

@bobis33 bobis33 commented Dec 18, 2025

No description provided.

@bobis33 bobis33 self-assigned this Dec 18, 2025

struct ShaderModuleDesc
{
// ShaderSourceType type;

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.

Copilot Autofix

AI 16 days ago

To fix the problem, simply remove the commented-out code on line 36: // ShaderSourceType type;. This will make the code cleaner, less confusing, and prevent the risk of misleading developers or creating unnecessary clutter. Only edit the affected region, leaving the remainder of the struct and file unchanged. No further changes, imports, methods, or definitions are necessary.

Suggested changeset 1
modules/Interfaces/include/Interfaces/Renderer/IShader.hpp

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/modules/Interfaces/include/Interfaces/Renderer/IShader.hpp b/modules/Interfaces/include/Interfaces/Renderer/IShader.hpp
--- a/modules/Interfaces/include/Interfaces/Renderer/IShader.hpp
+++ b/modules/Interfaces/include/Interfaces/Renderer/IShader.hpp
@@ -33,7 +33,6 @@
 
 struct ShaderModuleDesc
 {
-    // ShaderSourceType type;
     ShaderID id;
     std::string source;
     ShaderStage stage;
EOF
@@ -33,7 +33,6 @@

struct ShaderModuleDesc
{
// ShaderSourceType type;
ShaderID id;
std::string source;
ShaderStage stage;
Copilot is powered by AI and may make mistakes. Always verify output.
from application/engine to renderer plugin without hard coded shaders in renderer
@bobis33 bobis33 force-pushed the Feature/Plugin/Shaders branch 2 times, most recently from 0b8417c to 88dc66b Compare December 19, 2025 15:51
@bobis33 bobis33 force-pushed the Feature/Plugin/Shaders branch from 88dc66b to fa43b80 Compare December 23, 2025 10:49
@bobis33 bobis33 force-pushed the Feature/Plugin/Shaders branch from 7cf894b to d57fa27 Compare December 24, 2025 15:12
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.

2 participants