getHex/setHex are broken for ofFloatColor/ofShortColor

[kylemcdonald]

one solution is to pre-scale or post-scale the inputs/outputs using the limit() method for division/multiplication.

the best solution would also maintain the speed of the current implementation for ofColor, and not do any unnecessary scaling in that case.

[arturoc]

what about making a specialization for ofColor?

template<>
float ofColor_<unsigned char> getHsb(){
   //non-scaled version
}

[kylemcdonald]

that's really smart. then you could wrap it by casting to ofColor for the other cases, right?

template<class PixelType>
int ofColor_<PixelType>::getHex(){
    return ((ofColor) *this)::getHex();
}
template<>
int ofColor_<unsigned char>::getHex(){
    //non-scaled version
}

[arturoc]

yeah i was actually thinking on doing 2 versions but that is even better

[arturoc]

mmh, actually you'll need to do some post scaling, if you'll be returning an ofColor_ it will work automatically but since it's a number you'll need to scale it before returning but the rest should work

[thiagohersan]

closed by #1814