Matroska-Org / libmatroska

a C++ libary to parse Matroska files (.mkv and .mka)
GNU Lesser General Public License v2.1
318 stars 57 forks source link

[RFC] keep the min/max/webm/divx information for each element #148

Closed robUx4 closed 7 months ago

robUx4 commented 8 months ago

Based on https://github.com/Matroska-Org/libebml/pull/207

For now it's only informational but it can be used later for a libmatroska-based mkvalidator/mkclean. The way it's handled may evolve given how I manage to recode these tools.

mbunkus commented 7 months ago

This branch doesn't merge with the current master at the moment. This prevents me from reviewing the corresponding libEBML PRs (Matroska-Org/libebml#207, Matroska-Org/libebml#211). Can you please rebase it (or probably regenerate KaxSemantic.cpp, probably faster)? Thanks.

robUx4 commented 7 months ago

I rebased this branch, the libmatroska one and the spectools one.

mbunkus commented 7 months ago

I did merge & push the branch… not sure why Github thinks I didn't?

robUx4 commented 7 months ago

Indeed, that's weird but the code seems to be OK.