OpenCV flip - Cinder-KCB (SDK 1.8 Kinect Xbox360 )

[okinp]

Thanks a lot @lithium & @paul.houx !!

I tried using ip::horizontalFlip and it kind of works… but I found a bug in Cinder.

Surface->hasAlpha() returns false for SurfaceChannelOrder::BGRX (mCode = 5) which makes the ip::horizontalFlip use an increment of 3 bytes instead of 4.

So basically wherever in Cinder “hasAlpha” is used to decide what the byte increment should be, for specific SurfaceChannelOrders ( I suspect RGBX, BGRX, XRGB, XBGR) there will be a mistake.

Thanks again!

edit: I just posted the bug on a separate thread:

[Bug] ip::horizontalFlip

Hello, I just found a bug in ip::flipHorizontal. Surface.hasAlpha() is used to decide the pixel byte increment. In the case of BGRX hasAlpha returns false, but the increment should be 4 and not 3. I am not sure where else hasAlpha is used to decide the byte increment and I am not sure if making hasAlpha return true for BGRX would break something else… @andrewfb ? I guess the following should fix the problem in the case of flipHorizontal but not anywhere else: template<typename T> void fli…