ColorCorrection.h: Typo, variable undefined
- LordBelakor
- Topic Author
This is the line:
if ( Use_COLORSAT == 0 ) colorSat = 1.0f;
I tried installing a Witcher 3 preset, and got a undefined variable exception leading me to this point in the file.
I changed it to this to work:
if ( USE_COLORSAT == 0 ) colorSat = 1.0f;
The mistake is located in \ReShade 0.19.0 Public Beta with Framework\ReShade\CustomFX\ColorCorrection.h at line 327.
Next time maybe a bit more care when working with case sensitive variables
Please Log in or Create an account to join the conversation.
- crosire
I'll pass this along to Lucifer, who's managing those shaders. Thanks for reporting!
Please Log in or Create an account to join the conversation.
- Ganossa
USE is a keyword for shader. COLORSAT is only a "boolean" variable of a shader. Therefore, the variable got intenionally renamed ( with specifically a lot of care for case sensitivity )
Please Log in or Create an account to join the conversation.
- LordBelakor
- Topic Author
LuciferHawk wrote: This is no mistake.
USE is a keyword for shader. COLORSAT is only a "boolean" variable of a shader. Therefore, the variable got intenionally renamed ( with specifically a lot of care for case sensitivity )
The mistake was writing "Use" instead of "USE". I'm no expert but considering every other similarly structured variable has uppercase "USE", and having lowercase "Use" breaks old presets, this has to be a mistake. Unless you are expecting Users to change all their old presets for only this one variable from now on. I'd say its less work just bundling this little change into the next update to ensure compatibility with old Presets.
Please Log in or Create an account to join the conversation.
- Ioxa
Like the LuciferHawk said, it was not a mistake. It needed to be changed for compatibility with the Mediator. And thats not the only one that was changed. Unfortunately it does break old presets so you will have to carry those values over manually. There may be a way to make the shaders compatible with old presets but the old presets would not be compatible with the Mediator.LordBelakor wrote:
LuciferHawk wrote: This is no mistake.
USE is a keyword for shader. COLORSAT is only a "boolean" variable of a shader. Therefore, the variable got intenionally renamed ( with specifically a lot of care for case sensitivity )
The mistake was writing "Use" instead of "USE". I'm no expert but considering every other similarly structured variable has uppercase "USE", and having lowercase "Use" breaks old presets, this has to be a mistake. Unless you are expecting Users to change all their old presets for only this one variable from now on. I'd say its less work just bundling this little change into the next update to ensure compatibility with old Presets.
Please Log in or Create an account to join the conversation.