search

cambackup / m3u8-generator

m3u8 Generator
7 stars 7 forks source link

Spaces in playlist files? #2

Open westurner opened 6 years ago

westurner commented 6 years ago

Is there a reason that spaces are removed from playlist entries? https://github.com/cambackup/m3u8-generator/blob/6197814d579e4ff62802ead62d998423747cc98a/m3u8_generator/__init__.py#L27

As well, += creates string copies for each operation. list.join('\n') does not make intermediate copies.

I've also added these comments on the initial commit: https://github.com/cambackup/m3u8-generator/commit/44d0e4d9282558a24e4aa2aaca08c001c1677143#r27308236

westurner commented 6 years ago

This shows spaces in m3u playlists for local file names: https://en.wikipedia.org/wiki/M3U

If URIs need the spaces to be url-encoded?:

(scheme, netloc, path, query, fragment) = urlparse.urlsplit(uri)
if scheme:
    uri = urllib.quote(uri)  # urllib.quote_plus(uri)