Opened 4 years ago

Closed 3 years ago

#4347 closed enhancement (fixed)

support writing crc elements to level 1 elements of matroska

Reported by: dericed Owned by:
Priority: wish Component: avformat
Version: git-master Keywords: mkv bounty
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the enhancement request:

The Matroska specification states "All level 1 elements should include a CRC-32." ​http://matroska.org/technical/specs/index.html

I propose supporting an option to add CRC sub-elements while muxing to MKV. Just a dream.

Change History (14)

comment:1 Changed 4 years ago by gjdfgh

What for? Is there even anything that checks these CRC elements?

comment:2 Changed 4 years ago by dericed

ACtually I'm not sure and do have the same question. However based on the upcoming work at the Preforma project, ​http://preforma-project.eu/, I think the development of a Matroska implementation checker would feasibly cover this feature. The use of Matroska within a digital preservation context makes the CRC feature very interesting.

comment:3 Changed 4 years ago by dericed

  • Summary changed from support write crc elements to level 1 elements of matroska to support writing crc elements to level 1 elements of matroska

comment:4 Changed 4 years ago by dericed

Anyone interested in working on adding crc support to the mkv muxer if this can be sponsored?

comment:5 Changed 4 years ago by dericed

Here is a sample Matroska file with Level 1 CRC Elements. ​http://archive.org/download/159753000/blackpool_1-0_manutd-regmo.blogspot.com.mkv

comment:6 Changed 3 years ago by dericed

mediainfo can now verify EBML CRC-32 elements in Matroska. I also note that mkvclean can write level 1 CRC-32 elements while re-muxing.

comment:7 Changed 3 years ago by cehoyos

  • Keywords mkv added; matroska crc removed

comment:8 Changed 3 years ago by dericed

  • Keywords bounty added

comment:9 follow-up: Changed 3 years ago by dericed

Offering $600 bounty on this feature.

Last edited 3 years ago by dericed (previous) (diff)

comment:10 in reply to: ↑ 9 Changed 3 years ago by jamrial

Replying to dericed:

Offering $600 bounty on this feature.

Alright, I'll tackle this.

comment:11 follow-up: Changed 3 years ago by dericed

Thanks @jamrial, looking forward to testing it.

comment:12 in reply to: ↑ 11 Changed 3 years ago by jamrial

Replying to dericed:

Thanks @jamrial, looking forward to testing it.

I just sent the set to the mailing list.

comment:14 Changed 3 years ago by dericed

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.