mpv-player / mpv-player/mpv

Support for `#EXTINF:` key-value pairs variant

Open
#11,083 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

meta:feature-request
Dominant language
C
Stars
37k
Forks
3.5k
Avg merge
1d 10h
Merged PRs (30d)
22

Description

mpv 0.35+git.20221222.657fd280

Reproduction steps

Save this content as playlist.m3u and open it with mpv:

#EXTM3U
#EXTINF:-1 tvg-name="Hi, how are you",HI, HOW ARE YOU
https://edisciplinas.usp.br/pluginfile.php/5196097/mod_resource/content/1/Teste.mp4
Expected behavior

media-title should be HI, HOW ARE YOU.

Actual behavior

media-title is how are you",HI, HOW ARE YOU because mpv doesn't escape commas in key-value pairs.

Didn't find official m3u specification, but key-value pairs in #EXTINF: lines are popular. Even Wikipedia mentions it.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the issue by saving the supplied playlist.m3u and opening it with mpv, then trace the playlist parser handling #EXTINF: lines and key-value pairs. Done means the media-title is parsed as "HI, HOW ARE YOU" rather than including the key-value text after the comma.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.