Replaces the unwrap() call that causes the panic (when called on an Option with value None) with an and_then() call that calls decode_head_frame() and converts the Result to an Option.
Fixes two build warnings for warning: unused borrow that must be used.
Note: To get this to build, you'll likely need to merge #14 first, and then merge up master to the select-stream branch before merging this PR
With the above change, I test a few streams and got identical output with these changes, mlp v0.5.0, and DGDemux. So, in the case where the entire TrueHD stream for a given playlist is contained within a single segment, it seems like this is working (and DGDemux may have fixed previous differences in its processing?).
domyd
commented
2 years ago
Closes #15.
unwrap()call that causes the panic (when called on anOptionwith valueNone) with anand_then()call that callsdecode_head_frame()and converts theResultto anOption.warning: unused borrow that must be used.masterto theselect-streambranch before merging this PRWith the above change, I test a few streams and got identical output with these changes, mlp v0.5.0, and DGDemux. So, in the case where the entire TrueHD stream for a given playlist is contained within a single segment, it seems like this is working (and DGDemux may have fixed previous differences in its processing?).