search

intel / libyami

Yet Another Media Infrastructure. it is core part of media codec with hardware acceleration, it is yummy to your video experience on Linux like platform.
Apache License 2.0
146 stars 106 forks source link

Fix guess resolution overflow #838

Closed xuguangxin closed 6 years ago

xuguangxin commented 6 years ago

Instead of use a088df6, I think this fix may be better.

  1. It avoids change with and heigh to long long in https://github.com/intel/libyami/pull/836/files#r178894788.
  2. it's inplace operation, avoid the new allocation string object
xuguangxin commented 6 years ago

@uartie , Sorry it reverted by the c8069f8c2d04b6706b88e963b1694c5c5f339225. I added it back.

xuguangxin commented 6 years ago

@uartie , done, you read code so carefully. thanks