dirty fix for #3404

[like-a-bause]

new setFromAlignedPixels gets taken when PixelFormat == OF_PIXELS_I420.

[arturoc]

thanks, can you add a check for the format and number of samples, something like:

switch(pixelFormat){
case OF_PIXELS_I420:
if(samples.size() != 3){
    ofLogError("ofPixels") << "number of planes for I420 should be 3";
}
...
break;
default:
    ofLogError("ofPixels") << "setFromAlignedPixels with planes strides: pixel format not supported yet";
break;
}

also if we could have an option to send anything without planes to the simpler setFromAligned liek:

case OF_PIXELS_RGB:
case OF_PIXELS_RGBA:
case OF_PIXELS_GRAY:
case OF_PIXELS_GRAY_ALPHA:
    setFromAlignedPixels(pixels,w,h,pixelFormat,strides[0]);

[arturoc]

this doesn't make much sense for I420 since it doesn't have channels. something like:

if(width==strides[0] && width/2==strides[1] && width/2==strides[2])

would be a better check

[like-a-bause]

i added the stride size checks. Should the format and number checks go into ofPixels::setFromAlignedPixels or ofGstUtils::process_sample? How do I get samples.size()? Or do you mean strides.size() ?

[like-a-bause]

Ok i assume you meant strides.size() != 3
implemented that.

[arturoc]

yeah sorry i meant strides, looks great now thanks! i'll let the tests run and then merge it

[arturoc]

thanks!