search

DzenanJupic / rustube

A YouTube video downloader, originally inspired by pytube, written in Rust.
Apache License 2.0
227 stars 41 forks source link

[DRAFT]: add option for writing to stdout #60

Open mustafasegf opened 1 year ago

mustafasegf commented 1 year ago

right now this pr is a draft.

rustube right now only able to write to file. This pr added the ability to write to async write trait.

solves #59

mustafasegf commented 1 year ago

this is more of a proof of concept (a working one). I do have some question regarding how to implement this. Since a lot of the function are tightly coupled to path, there's a lot of rewrite need to be done. Should i make another set of funtion just for writing to async write so the original public facing api doesnt change or should i rewrite it to use async write only, but we will loose the ability to give information about whre the file is donwloaded.