Feature request: Take VideoOptions by reference

Mithronn / rusty_ytdl

A Rust library for Youtube video searcher and downloader

https://docs.rs/rusty_ytdl

MIT License

107 stars 20 forks source link

Closed nick42d closed 2 months ago

nick42d commented 3 months ago

Just a small one, logging it whilst I remember.

Would be nice if Video::new_with_options() would take VideoOptions by reference.

I am storing my VideoOptions in a struct so I can re-use it when running concurrent downloads.

Happy to take a look and raise a PR if there are no technical issues.

Mithronn commented 3 months ago

There don't seem to be any technical problems. You can raise PR.