search

intel / libvpl

Intel® Video Processing Library (Intel® VPL) API, dispatcher, and examples
https://intel.github.io/libvpl/
MIT License
262 stars 80 forks source link

Fixed the issue that redundant pixel(s) would be written to output file. #67

Closed Austin-Hu closed 1 year ago

Austin-Hu commented 1 year ago

Fixes Issue #66.

When the bitstream was encoded with width or height aligned.

Signed-off-by: Austin Hu austin.hu@intel.com

Austin-Hu commented 1 year ago

Hi @jeffreymcallister,

Please help review this PR with Issue #66 if you'd be free.

Thanks!

jeffreymcallister commented 1 year ago

Sorry for the delayed reply. We have reviewed your fix and it works. One request before merging though: as you may have seen, each example has a util.h file. We want to keep util.h identical for every example. Would you prefer to copy your fixed util.h to the other examples and update the PR, or would you rather that we do it?

Austin-Hu commented 1 year ago

@jeffreymcallister, no worries. Thanks for your review! And could you help make the change to other util.h files and push more commits to this PR? Because definitely you'd have more cases to verify for code quality. Thanks!

SupreetSinghPalne commented 1 year ago

Hi Austine, this issue is fixed and merged. Commit Id - e9990fe

Austin-Hu commented 1 year ago

Hi Austine, this issue is fixed and merged. Commit Id - e9990fe

Thanks! Let me close this PR.