[FFmpeg-devel] [PATCH] dpx: fix some stupid typos

[Georg Lippitsch]

Am 28.08.2012, 16:05 Uhr, schrieb Paul B Mahol <onemda at gmail.com>:

>>          case 16:
>>              if (planar) {
>> -                int source_bpc = target_packet_size / elements;
>> +                int source_bpc = source_packet_size / elements;
>>                  int target_bpc = target_packet_size / elements;
>>                  for (x = 0; x < avctx->height; x++) {
>>                      uint8_t *dst[AV_NUM_DATA_POINTERS];
>>                      for (i=0; i<elements; i++)
>>                          dst[i] = ptr[i];
>>                      for (y = 0; y < avctx->width; y++) {
>> -                        for (i=0; i<3; i++) {
>> +                        for (i=0; i<elements; i++) {
>
> Shouldn't this crash if there is alpha channel and gbrp is used? gbrp
> only have 3 components.

Indeed. I also found out that there are some more flaws with the 12 Bit  
decoder (but 10 Bit seem to work well).
I'll submit some more patches in the next days.


Georg