Skip to content

Use R"()" syntax for glsl shader strings for better readability #4795

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.

Sign up for GitHub

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

Jump to bottom
Merged
lgritz merged 1 commit into from
Jun 6, 2025
Merged

Use R"()" syntax for glsl shader strings for better readability #4795

lgritz merged 1 commit into from
Jun 6, 2025

Conversation

@soswow
Copy link
Contributor

@soswow soswow commented Jun 6, 2025

Small PR to improve redability of shader code parts.

@soswow soswow changed the title Use R"()" syntax for glsl shader strings for better visibility Use R"()" syntax for glsl shader strings for better readability Jun 6, 2025
lgritz
lgritz approved these changes Jun 6, 2025
View reviewed changes
Copy link
Collaborator

@lgritz lgritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much nicer, thanks!

@lgritz lgritz merged commit 61227e3 into AcademySoftwareFoundation:main Jun 6, 2025
31 checks passed
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request Jun 6, 2025
@soswow @lgritz
Use R"()" syntax for glsl shader strings for better readability (Acad…
eb75878 
…emySoftwareFoundation#4795)

Small PR to improve redability of shader code parts.

Signed-off-by: Aleksandr Motsjonov <[email protected]>
zachlewis pushed a commit to zachlewis/OpenImageIO that referenced this pull request Aug 1, 2025
@soswow @zachlewis
Use R"()" syntax for glsl shader strings for better readability (Acad…
7872f93 
…emySoftwareFoundation#4795)

Small PR to improve redability of shader code parts.

Signed-off-by: Aleksandr Motsjonov <[email protected]>
@BrewTestBot BrewTestBot mentioned this pull request Oct 2, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lgritz lgritz lgritz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@soswow @lgritz