RSully / RSColorPicker

iOS color picker view with brightness control, opacity control, and delegation support.
Other
302 stars 77 forks source link

Replace otherwise unused variable with function call in assertions #39

Closed rhwood closed 10 years ago

rhwood commented 10 years ago

Since the height variable is only used inside an NSAssert, and NSAsserts are disabled by default in Release builds, this change prevents four warnings (two standard, two static analysis) against a product build.

RSully commented 10 years ago

Thanks!

I guess it is technically possible for a release build to enable them. I hadn't considered whether this was done by default or not.


Per Apple's doc:

Assertions are disabled if the preprocessor macro NS_BLOCK_ASSERTIONS is defined [...] Note: Not all release configurations disable assertions by default

rhwood commented 10 years ago

Thanks.