Event label fields (&el=...)

[0xced]

The YouTube extractor tries 4 different el types.

I noticed that embedded and detailpage are enough for all the videos I tested. Trying vevo or empty never turned up any result that embedded and detailpage would not get. Is this really necessary to try all four? Can you point me to a video that would work only if trying with vevo or empty?

[phihag]

Taking out vevo or '' would do not have an extremely negligible impact on runtime, because the further types are only evaluated once the first two fail. Therefore, why do you think the additional types pose a problem in the first place?

Unfortunately, I cannot find the original issues - I believe they were lost in the move from bitbucket to GitHub (and 787f2a5) doesn't mention an example URL. Since the further tries should not hurt, I don't see any reason for removing them though. If you can present a reason, I'll be happy to remove them.