Skip to content

/ youtube-dl

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reuse the instances of InfoExtractors in YoutubeDL #998

Closed
jaimeMF opened this issue Jul 7, 2013 · 0 comments
Closed

Reuse the instances of InfoExtractors in YoutubeDL #998

jaimeMF opened this issue Jul 7, 2013 · 0 comments

Comments

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jul 7, 2013

Currently, when it gets an url result with the ie_key set it creates a new instance of the extractor. Normally this is not bad, but if the extractor is YoutubeIE, it would need to be initialize again and if it have to login it'll waste time, since it's already done.

This happens for example when downloading the Youtube subscriptons feed, it could also happen if support for private playlists is added.

I think that instead of creating a new instance, we should reuse them.
@jaimeMF jaimeMF closed this in 56c7366 Aug 28, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@jaimeMF
You can’t perform that action at this time.