search

monsieurvideo / get-flash-videos

Download or play videos from various Flash-based video hosting sites, without having to use the Flash player.
Apache License 2.0
240 stars 68 forks source link

Patch for issue 197 HLSDownloader memory usage to high #198

Closed pcwalden closed 8 years ago

pcwalden commented 8 years ago

HLSDownloader use of browser object was storing each TS segment in memory and not releasing the memory. With large video files, the process memory could grow large enough that the kernel would kill it.

This patch diverts each downloaded segment to a re-used segment file and then appends the segment to the TS file. The process memory size no longer grows.

karjonas commented 8 years ago

Looks good and works on the sites I tried.

njtaylor commented 8 years ago

Thanks for the update.