-
Notifications
You must be signed in to change notification settings - Fork 3
Support vertex attribute and uniform parsing with structured types, built-in attributes, conditional compilation, and variable expansion tracking at crates/jsbindings/webgl #393
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
Closed
Closed
Changes from 14 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
6393a21
Initial plan
Copilot e3bbc26
Remove deprecated concat_idents feature flag
Copilot 82ef186
Implement GLSL attribute parser with FFI bindings
Copilot a21a221
Add C++ wrapper for GLSL attribute parser FFI
Copilot 304c484
Add comprehensive documentation for GLSL attribute parser
Copilot d099cea
Fix C++ code formatting with clang-format
Copilot 7293390
Revert accidental formatting change in all_events.cpp
Copilot 1710d2f
update
yorkie 8a6189e
Refactor to use JSON serialization for GLSL attributes FFI
Copilot 2408daa
update
yorkie 5712fc9
Merge branch 'main' of github.com:M-CreativeLab/jsar-runtime into cop…
yorkie 7e3a373
fix
yorkie 1aef9bf
fix
yorkie a8a3c7b
Filter out inactive/unreferenced attributes from parser results
Copilot de2c8f7
Add uniform parsing support and rename to GLSLShaderAnalyzer
Copilot fa70daa
Use GL type enums, add size and active fields
Copilot 9a4ebbb
Add combined Parse() method to parse attributes and uniforms in singl…
Copilot 70ad563
Implement structured uniform support with array and nested struct exp…
Copilot bb091fc
Add support for gl_InstanceID/gl_VertexID as active attributes
Copilot 0713c17
fix
yorkie a120c94
Add comprehensive tests for preprocessor and array size handling
Copilot 1c26a6e
Add test for conditional compilation with #ifdef directives
Copilot 5774f55
update
yorkie 617df18
Fix single-element array notation and add variable expansion tracking
Copilot 9ee327a
Fix stack overflow in variable alias tracking
Copilot 5870ccd
update
yorkie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot A single invocation of
parseshould produce the parsedattribsanduniforms, rather than calling it twice.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added combined
Parse()method that parses shader once and returns both attributes and uniforms. CreatedGLSLShaderVariablesstruct andparseGLSLShaderFFI function. The separateParseAttributes()andParseUniforms()methods are now marked as deprecated. (commit 9a4ebbb)