Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Update getMediaInformation implementation using JSON parsing #417

Closed
tanersener opened this issue May 8, 2020 · 2 comments
Closed

Update getMediaInformation implementation using JSON parsing #417

tanersener opened this issue May 8, 2020 · 2 comments
Assignees
@tanersener
Labels
enhancement New feature or request fixed
Projects
The Next FFmpeg Release

Comments

@tanersener
Copy link
Owner

tanersener commented May 8, 2020 •
edited

Current approach uses line parsing which is prone to errors. Since FFprobe supports JSON, updating getMediaInformation implementation to use JSON will solve these errors and will allow flexibility in terms of getting uncommon properties.

  • Update parsing implementation
  • Allow media information extraction using custom ffprobe command
@tanersener tanersener added the enhancement New feature or request label May 8, 2020
@tanersener tanersener self-assigned this May 8, 2020
@tanersener tanersener added this to To do in The Next FFmpeg Release via automation May 8, 2020
@tanersener tanersener moved this from To do to In progress in The Next FFmpeg Release May 9, 2020
tanersener added a commit that referenced this issue May 9, 2020
@tanersener
Use JSON parsing in getMediaInformation, fixes #417
7b80cd3
@tanersener tanersener moved this from In progress to Done in The Next FFmpeg Release May 9, 2020
@tanersener
Copy link
Owner Author

tanersener commented May 9, 2020 •
edited

Fixed in the development branch.

@tanersener tanersener mentioned this issue May 10, 2020
Closed
@tanersener
Copy link
Owner Author

Released in v4.4.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@tanersener tanersener

Labels
enhancement New feature or request fixed
Projects
No open projects
The Next FFmpeg Release
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tanersener