MediaInfoLib icon indicating copy to clipboard operation
MediaInfoLib copied to clipboard

File__analyze_streams.cpp - Change for 2.39:1 DisplayAspectRatio/Stri…

Open kieranjol opened this issue 7 years ago • 2 comments

…ng #612

This attempts to fix #612. I haven't tested this yet but I wanted to see if you thought this was sane.. It looks like two of us have a use case to have 2.39 display as intended anyhow..

kieranjol avatar Mar 31 '18 16:03 kieranjol

I wanted to see if you thought this was sane..

It is sane, sure, and IMO it is bug fix so "breaking" (it should break nothing as I expect people to use the raw vvalue) old stuff is fine.

I let you both debate while I enjoy some days AFK :).

JeromeMartinez avatar Apr 02 '18 09:04 JeromeMartinez

Ha! Enjoy the AFK - I think @Sami32 and I both agree on the content of this particular pull request anyhow, perhaps the other comments can be resolved in the other PR by @Sami32 ?

kieranjol avatar Apr 02 '18 09:04 kieranjol