From 63108b883593fe427a93c545f6ea6fb7060103e2 Mon Sep 17 00:00:00 2001
From: John Hawkinson <jhawk@mit.edu>
Date: Tue, 7 Mar 2017 07:52:08 -0500
Subject: [PATCH] [brightcove:new] flake8 for PEP 8

I really reget not being able to use this compact syntax, as well as to align
with spaces:
            if not account_id: account_id  = attrs.get('data-account')
            if not player_id:  player_id   = attrs.get('data-player')
            if not embed:      embed       = attrs.get('data-embed')
but it really upsets flake8:

brightcove.py:519:30: E701 multiple statements on one line (colon)
brightcove.py:519:42: E221 multiple spaces before operator
brightcove.py:520:29: E701 multiple statements on one line (colon)
brightcove.py:520:30: E241 multiple spaces after ':'
brightcove.py:520:41: E221 multiple spaces before operator
brightcove.py:521:25: E701 multiple statements on one line (colon)
brightcove.py:521:26: E241 multiple spaces after ':'
brightcove.py:521:37: E221 multiple spaces before operator
---
 youtube_dl/extractor/brightcove.py | 35 +++++++++++++++++-------------
 1 file changed, 20 insertions(+), 15 deletions(-)

diff --git a/youtube_dl/extractor/brightcove.py b/youtube_dl/extractor/brightcove.py
index 9f63ab79c..31a947f4c 100644
--- a/youtube_dl/extractor/brightcove.py
+++ b/youtube_dl/extractor/brightcove.py
@@ -487,7 +487,7 @@ class BrightcoveNewIE(InfoExtractor):
         # 5. https://support.brightcove.com/en/video-cloud/docs/dynamically-assigning-videos-player
 
         # [1] looks like:
-        # 	<video data-video-id="5320421710001" data-account="245991542" data-player="SJWAiyYWg" data-embed="default" class="video-js" controls itemscope itemtype="http://schema.org/VideoObject">
+        # <video data-video-id="5320421710001" data-account="245991542" data-player="SJWAiyYWg" data-embed="default" class="video-js" controls itemscope itemtype="http://schema.org/VideoObject">
 
         entries = []
 
@@ -498,28 +498,33 @@ class BrightcoveNewIE(InfoExtractor):
 
         # Look for <video> tags [1] and embed_in_page embeds [2]
         for video, script_tag, account_id, player_id, embed in re.findall(
-        r'''(?isx)
-            (<video[^>]+>)
-            (?:.*?
-              (<script[^>]+
-                src=["\'](?:https?:)?//players\.brightcove\.net/
-                (\d+)/([^/]+)_([^/]+)/index(?:\.min)?\.js
-              )
-            )?
-        ''', webpage):
+            r'''(?isx)
+                (<video[^>]+>)
+                (?:.*?
+                  (<script[^>]+
+                    src=["\'](?:https?:)?//players\.brightcove\.net/
+                    (\d+)/([^/]+)_([^/]+)/index(?:\.min)?\.js
+                  )
+                )?
+            ''', webpage
+        ):
             attrs = extract_attributes(video)
 
             # According to examples from [4] it's unclear whether video id
             # may be optional and what to do when it is
-            video_id    = attrs.get('data-video-id')
+            video_id = attrs.get('data-video-id')
             # See PR#12099/bostonglobe.py for 'data-brightcove-video-id' variant
 
-            if not account_id: account_id  = attrs.get('data-account')
-            if not player_id:  player_id   = attrs.get('data-player')
-            if not embed:      embed       = attrs.get('data-embed')
+            if not account_id:
+                account_id = attrs.get('data-account')
+            if not player_id:
+                player_id = attrs.get('data-player')
+            if not embed:
+                embed = attrs.get('data-embed')
 
             # According to [5] data-video-id may be prefixed with 'ref:'
-            if video_id: video_id = video_id.rpartition('ref:')[2]
+            if video_id:
+                video_id = video_id.rpartition('ref:')[2]
 
             if video_id and account_id and player_id and embed:
                 entries.append(