libretro / slang-shaders

213 stars 91 forks source link

Fishcu/authentic gbc #582

Closed fishcu closed 2 months ago

fishcu commented 2 months ago

Sample screenshots on Discord: https://discord.com/channels/184109094070779904/662208336984277012/1234234573265571913

hizzlekizzle commented 2 months ago

Is there a reason you decided to split the parameters and 'common' parts into separate files? If so, and you don't want to merge them into the main shader, would you mind at least changing their file extension to .inc or .h or whatever so nobody tries to load them as a standalone slang shader passes (since they're not)?

fishcu commented 2 months ago

Yes, the reason I split out these files are as follows:

A survey I did back in August 2023 showed that the file endings used are pretty mixed: https://discord.com/channels/184109094070779904/584075942276628481/1136000493617958932 I don't really want to use *.h since that implies a C header. I may change the file endings to *.inc if that helps to prevent users from opening the files. I should point out that I have been doing the inclusion of files with *.slang file endings, which are not stand-alone shaders, for some other presets as well, and have not gotten any reports of errors.

hizzlekizzle commented 2 months ago

Ah, yeah, I hadn't noticed that those weren't functional slang shaders when I merged them. You can give them whatever extension you like, but if it's going to be *.slang in a repo of slang shaders, it should be compileable as a slang shader.

fishcu commented 2 months ago

OK, I'll fix the files in this PR, then make another PR to fix the other files that are affected by this.