- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
It seems there is a bug while encoding chroma components in the H.264 encoder.
The function H264CoreEncoder_EncodeChroma encode in one shot both the U and V components. It starts with the U and upon reaching the first block of the V components switch pointers to the V plane. However, the condition to switch components doesn't seem right and upon reaching block 2 of the V components the U pointer are used again.
Emmanuel
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
Emmanuel, the "do" loop has only 2 iterations, so that uBlock is first U (==16) at first and then firstV (==16+num_blocks[chroma]) at second time. There are few different block loops inside do{}, like
for (uBlock = startBlock; uBlock < uLastBlock; uBlock ++)
Regards,
Leonid
Link Copied
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
thanks for reporting on this. Could you please specify what version of IPP do you use? Just worrying if you had a chance to try the latest IPP 6.1 update 1 version?
Regards,
Vladimir
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
thanks for reporting on this. Could you please specify what version of IPP do you use? Just worrying if you had a chance to try the latest IPP 6.1 update 1 version?
Regards,
Vladimir
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
Vladimir
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
Emmanuel, the "do" loop has only 2 iterations, so that uBlock is first U (==16) at first and then firstV (==16+num_blocks[chroma]) at second time. There are few different block loops inside do{}, like
for (uBlock = startBlock; uBlock < uLastBlock; uBlock ++)
Regards,
Leonid
- Mark as New
- Bookmark
- Subscribe
- Mute
- Subscribe to RSS Feed
- Permalink
- Report Inappropriate Content
Vladimir
Sorry for the fast alarm. I have modified this part of the code on my side and though I was looking at the original Intel code.
Emmanuel
- Subscribe to RSS Feed
- Mark Topic as New
- Mark Topic as Read
- Float this Topic for Current User
- Bookmark
- Subscribe
- Printer Friendly Page