Ungodly mess - separated mediainfo

[qfjp]

The code is horrible and ad-hoc. The first step to fixing this is to unify the four mediainfo functions while adding Enumerations for everything possible within a video file's JSON.

• [ ] Unify StreamType
  • [x] mediainfo_<typ> functions now are one function taking typ as a parameter: mediainfo(fpath: PosixPath, typ: StreamType)
  • [x] All other functions dependent on the stream type take a StreamType enumerated object as a parameter (validate_conversions(..., typ: StreamType, ...), get_stream_ix_offset(...,typ: StreamType), generate_conversions(...,typ: StreamType,...)
  • [ ] Unify StreamType objects by structuring the above functions as methods to the enumerated class.
• [ ] Unify Verification
  • [X] Get rid of two verification functions (compare_json_key and compare_json_diff). The function should know how to handle it based on the stream property's properties. Fixed in commit 67d8dde
  • [ ] Create enumerated types for json properties within a stream (loop over these to verify).

[qfjp]

Unifying items in metadata and the like has been partially accomplished:

1. Unifying the mediainfo functions was done in commit 67d8dde, where all mediainfo_<typ> functions have become mediainfo(..., typ: StreamIndex). That is, they actually take the type as the parameter instead of hardcoding it into the name of the function.
2. Generating the arguments to encode various streams for FFMPEG has been unified under generate_conversions(...typ: StreamType) in commit 60f1c09