Fix: Reassigning value before the old one has been used

PixInsight / PCL

PixInsight Class Library

http://pixinsight.com/developer/pcl/

Other

42 stars 21 forks source link

Fix: Reassigning value before the old one has been used #44

Closed tstibor closed 6 years ago

tstibor commented 6 years ago

This switch statement misses break after each variable assignment.

jconejero commented 6 years ago

Wow, this one has passed unnoticed for years. Thank you so much for detecting it.

The ICCProfile::SupportsRenderingIntent() member function is currently not being used in standard modules and the core application, so the bug has not had any practical consequences. But this one is a really ugly error and it's great to have it fixed. Thank you again!