{"action":"created","comment":{"url":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments/42324933","id":42324933,"diff_hunk":"@@ -70,6 +70,21 @@ def xml_to_format(fnode):\n             '_available': is_available,\n         }\n \n+    def xml_to_thumbnails(fnode):\n+        thumbnails = list()\n+        for node in fnode:\n+            thumbnail = {'url': node.text}\n+            if 'key' in node.attrib:\n+                if re.match(\"^[0-9]+x[0-9]+$\", node.attrib['key']):","path":"youtube_dl/extractor/zdf.py","position":9,"original_position":9,"commit_id":"b243340f0ce311443a15a2dfd4356a9504e18c04","original_commit_id":"b243340f0ce311443a15a2dfd4356a9504e18c04","user":{"login":"dstftw","id":1908898,"avatar_url":"https://avatars.githubusercontent.com/u/1908898?v=3","gravatar_id":"","url":"https://api.github.com/users/dstftw","html_url":"https://github.com/dstftw","followers_url":"https://api.github.com/users/dstftw/followers","following_url":"https://api.github.com/users/dstftw/following{/other_user}","gists_url":"https://api.github.com/users/dstftw/gists{/gist_id}","starred_url":"https://api.github.com/users/dstftw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dstftw/subscriptions","organizations_url":"https://api.github.com/users/dstftw/orgs","repos_url":"https://api.github.com/users/dstftw/repos","events_url":"https://api.github.com/users/dstftw/events{/privacy}","received_events_url":"https://api.github.com/users/dstftw/received_events","type":"User","site_admin":false},"body":"Since you use regex anyway you can just capture both numbers and use plain `int`. Also consistently use single quotes.","created_at":"2015-10-18T19:15:05Z","updated_at":"2015-10-18T19:15:05Z","html_url":"https://github.com/rg3/youtube-dl/pull/7225#discussion_r42324933","pull_request_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225","_links":{"self":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments/42324933"},"html":{"href":"https://github.com/rg3/youtube-dl/pull/7225#discussion_r42324933"},"pull_request":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225"}}},"pull_request":{"url":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225","id":48013607,"html_url":"https://github.com/rg3/youtube-dl/pull/7225","diff_url":"https://github.com/rg3/youtube-dl/pull/7225.diff","patch_url":"https://github.com/rg3/youtube-dl/pull/7225.patch","issue_url":"https://api.github.com/repos/rg3/youtube-dl/issues/7225","number":7225,"state":"open","locked":false,"title":"adds thumbnail support for ZDF Mediathek extractor","user":{"login":"kennell","id":2326287,"avatar_url":"https://avatars.githubusercontent.com/u/2326287?v=3","gravatar_id":"","url":"https://api.github.com/users/kennell","html_url":"https://github.com/kennell","followers_url":"https://api.github.com/users/kennell/followers","following_url":"https://api.github.com/users/kennell/following{/other_user}","gists_url":"https://api.github.com/users/kennell/gists{/gist_id}","starred_url":"https://api.github.com/users/kennell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennell/subscriptions","organizations_url":"https://api.github.com/users/kennell/orgs","repos_url":"https://api.github.com/users/kennell/repos","events_url":"https://api.github.com/users/kennell/events{/privacy}","received_events_url":"https://api.github.com/users/kennell/received_events","type":"User","site_admin":false},"body":"This commit adds thumbnail support to the ZDF Mediathek extractor","created_at":"2015-10-18T17:57:24Z","updated_at":"2015-10-18T19:15:05Z","closed_at":null,"merged_at":null,"merge_commit_sha":"00a31fa1b9bc1662fd9a2ece2d98b450d02554e3","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225/commits","review_comments_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225/comments","review_comment_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments{/number}","comments_url":"https://api.github.com/repos/rg3/youtube-dl/issues/7225/comments","statuses_url":"https://api.github.com/repos/rg3/youtube-dl/statuses/b243340f0ce311443a15a2dfd4356a9504e18c04","head":{"label":"kennell:master","ref":"master","sha":"b243340f0ce311443a15a2dfd4356a9504e18c04","user":{"login":"kennell","id":2326287,"avatar_url":"https://avatars.githubusercontent.com/u/2326287?v=3","gravatar_id":"","url":"https://api.github.com/users/kennell","html_url":"https://github.com/kennell","followers_url":"https://api.github.com/users/kennell/followers","following_url":"https://api.github.com/users/kennell/following{/other_user}","gists_url":"https://api.github.com/users/kennell/gists{/gist_id}","starred_url":"https://api.github.com/users/kennell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennell/subscriptions","organizations_url":"https://api.github.com/users/kennell/orgs","repos_url":"https://api.github.com/users/kennell/repos","events_url":"https://api.github.com/users/kennell/events{/privacy}","received_events_url":"https://api.github.com/users/kennell/received_events","type":"User","site_admin":false},"repo":{"id":44489198,"name":"youtube-dl","full_name":"kennell/youtube-dl","owner":{"login":"kennell","id":2326287,"avatar_url":"https://avatars.githubusercontent.com/u/2326287?v=3","gravatar_id":"","url":"https://api.github.com/users/kennell","html_url":"https://github.com/kennell","followers_url":"https://api.github.com/users/kennell/followers","following_url":"https://api.github.com/users/kennell/following{/other_user}","gists_url":"https://api.github.com/users/kennell/gists{/gist_id}","starred_url":"https://api.github.com/users/kennell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kennell/subscriptions","organizations_url":"https://api.github.com/users/kennell/orgs","repos_url":"https://api.github.com/users/kennell/repos","events_url":"https://api.github.com/users/kennell/events{/privacy}","received_events_url":"https://api.github.com/users/kennell/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/kennell/youtube-dl","description":"Small command-line program to download videos from YouTube.com and other video sites","fork":true,"url":"https://api.github.com/repos/kennell/youtube-dl","forks_url":"https://api.github.com/repos/kennell/youtube-dl/forks","keys_url":"https://api.github.com/repos/kennell/youtube-dl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/kennell/youtube-dl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/kennell/youtube-dl/teams","hooks_url":"https://api.github.com/repos/kennell/youtube-dl/hooks","issue_events_url":"https://api.github.com/repos/kennell/youtube-dl/issues/events{/number}","events_url":"https://api.github.com/repos/kennell/youtube-dl/events","assignees_url":"https://api.github.com/repos/kennell/youtube-dl/assignees{/user}","branches_url":"https://api.github.com/repos/kennell/youtube-dl/branches{/branch}","tags_url":"https://api.github.com/repos/kennell/youtube-dl/tags","blobs_url":"https://api.github.com/repos/kennell/youtube-dl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/kennell/youtube-dl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/kennell/youtube-dl/git/refs{/sha}","trees_url":"https://api.github.com/repos/kennell/youtube-dl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/kennell/youtube-dl/statuses/{sha}","languages_url":"https://api.github.com/repos/kennell/youtube-dl/languages","stargazers_url":"https://api.github.com/repos/kennell/youtube-dl/stargazers","contributors_url":"https://api.github.com/repos/kennell/youtube-dl/contributors","subscribers_url":"https://api.github.com/repos/kennell/youtube-dl/subscribers","subscription_url":"https://api.github.com/repos/kennell/youtube-dl/subscription","commits_url":"https://api.github.com/repos/kennell/youtube-dl/commits{/sha}","git_commits_url":"https://api.github.com/repos/kennell/youtube-dl/git/commits{/sha}","comments_url":"https://api.github.com/repos/kennell/youtube-dl/comments{/number}","issue_comment_url":"https://api.github.com/repos/kennell/youtube-dl/issues/comments{/number}","contents_url":"https://api.github.com/repos/kennell/youtube-dl/contents/{+path}","compare_url":"https://api.github.com/repos/kennell/youtube-dl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/kennell/youtube-dl/merges","archive_url":"https://api.github.com/repos/kennell/youtube-dl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/kennell/youtube-dl/downloads","issues_url":"https://api.github.com/repos/kennell/youtube-dl/issues{/number}","pulls_url":"https://api.github.com/repos/kennell/youtube-dl/pulls{/number}","milestones_url":"https://api.github.com/repos/kennell/youtube-dl/milestones{/number}","notifications_url":"https://api.github.com/repos/kennell/youtube-dl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/kennell/youtube-dl/labels{/name}","releases_url":"https://api.github.com/repos/kennell/youtube-dl/releases{/id}","created_at":"2015-10-18T17:49:41Z","updated_at":"2015-10-18T17:49:51Z","pushed_at":"2015-10-18T19:08:11Z","git_url":"git://github.com/kennell/youtube-dl.git","ssh_url":"git@github.com:kennell/youtube-dl.git","clone_url":"https://github.com/kennell/youtube-dl.git","svn_url":"https://github.com/kennell/youtube-dl","homepage":"http://rg3.github.io/youtube-dl/","size":144800,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"rg3:master","ref":"master","sha":"a6e0afa2bbc93d145b31911b8ce40c502994e2a1","user":{"login":"rg3","id":53487,"avatar_url":"https://avatars.githubusercontent.com/u/53487?v=3","gravatar_id":"","url":"https://api.github.com/users/rg3","html_url":"https://github.com/rg3","followers_url":"https://api.github.com/users/rg3/followers","following_url":"https://api.github.com/users/rg3/following{/other_user}","gists_url":"https://api.github.com/users/rg3/gists{/gist_id}","starred_url":"https://api.github.com/users/rg3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rg3/subscriptions","organizations_url":"https://api.github.com/users/rg3/orgs","repos_url":"https://api.github.com/users/rg3/repos","events_url":"https://api.github.com/users/rg3/events{/privacy}","received_events_url":"https://api.github.com/users/rg3/received_events","type":"User","site_admin":false},"repo":{"id":1039520,"name":"youtube-dl","full_name":"rg3/youtube-dl","owner":{"login":"rg3","id":53487,"avatar_url":"https://avatars.githubusercontent.com/u/53487?v=3","gravatar_id":"","url":"https://api.github.com/users/rg3","html_url":"https://github.com/rg3","followers_url":"https://api.github.com/users/rg3/followers","following_url":"https://api.github.com/users/rg3/following{/other_user}","gists_url":"https://api.github.com/users/rg3/gists{/gist_id}","starred_url":"https://api.github.com/users/rg3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rg3/subscriptions","organizations_url":"https://api.github.com/users/rg3/orgs","repos_url":"https://api.github.com/users/rg3/repos","events_url":"https://api.github.com/users/rg3/events{/privacy}","received_events_url":"https://api.github.com/users/rg3/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/rg3/youtube-dl","description":"Small command-line program to download videos from YouTube.com and other video sites","fork":false,"url":"https://api.github.com/repos/rg3/youtube-dl","forks_url":"https://api.github.com/repos/rg3/youtube-dl/forks","keys_url":"https://api.github.com/repos/rg3/youtube-dl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/rg3/youtube-dl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/rg3/youtube-dl/teams","hooks_url":"https://api.github.com/repos/rg3/youtube-dl/hooks","issue_events_url":"https://api.github.com/repos/rg3/youtube-dl/issues/events{/number}","events_url":"https://api.github.com/repos/rg3/youtube-dl/events","assignees_url":"https://api.github.com/repos/rg3/youtube-dl/assignees{/user}","branches_url":"https://api.github.com/repos/rg3/youtube-dl/branches{/branch}","tags_url":"https://api.github.com/repos/rg3/youtube-dl/tags","blobs_url":"https://api.github.com/repos/rg3/youtube-dl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/rg3/youtube-dl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/rg3/youtube-dl/git/refs{/sha}","trees_url":"https://api.github.com/repos/rg3/youtube-dl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/rg3/youtube-dl/statuses/{sha}","languages_url":"https://api.github.com/repos/rg3/youtube-dl/languages","stargazers_url":"https://api.github.com/repos/rg3/youtube-dl/stargazers","contributors_url":"https://api.github.com/repos/rg3/youtube-dl/contributors","subscribers_url":"https://api.github.com/repos/rg3/youtube-dl/subscribers","subscription_url":"https://api.github.com/repos/rg3/youtube-dl/subscription","commits_url":"https://api.github.com/repos/rg3/youtube-dl/commits{/sha}","git_commits_url":"https://api.github.com/repos/rg3/youtube-dl/git/commits{/sha}","comments_url":"https://api.github.com/repos/rg3/youtube-dl/comments{/number}","issue_comment_url":"https://api.github.com/repos/rg3/youtube-dl/issues/comments{/number}","contents_url":"https://api.github.com/repos/rg3/youtube-dl/contents/{+path}","compare_url":"https://api.github.com/repos/rg3/youtube-dl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/rg3/youtube-dl/merges","archive_url":"https://api.github.com/repos/rg3/youtube-dl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/rg3/youtube-dl/downloads","issues_url":"https://api.github.com/repos/rg3/youtube-dl/issues{/number}","pulls_url":"https://api.github.com/repos/rg3/youtube-dl/pulls{/number}","milestones_url":"https://api.github.com/repos/rg3/youtube-dl/milestones{/number}","notifications_url":"https://api.github.com/repos/rg3/youtube-dl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/rg3/youtube-dl/labels{/name}","releases_url":"https://api.github.com/repos/rg3/youtube-dl/releases{/id}","created_at":"2010-10-31T14:35:07Z","updated_at":"2015-10-18T18:52:31Z","pushed_at":"2015-10-18T19:12:44Z","git_url":"git://github.com/rg3/youtube-dl.git","ssh_url":"git@github.com:rg3/youtube-dl.git","clone_url":"https://github.com/rg3/youtube-dl.git","svn_url":"https://github.com/rg3/youtube-dl","homepage":"http://rg3.github.io/youtube-dl/","size":144800,"stargazers_count":12110,"watchers_count":12110,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":2479,"mirror_url":null,"open_issues_count":1330,"forks":2479,"open_issues":1330,"watchers":12110,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225"},"html":{"href":"https://github.com/rg3/youtube-dl/pull/7225"},"issue":{"href":"https://api.github.com/repos/rg3/youtube-dl/issues/7225"},"comments":{"href":"https://api.github.com/repos/rg3/youtube-dl/issues/7225/comments"},"review_comments":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225/comments"},"review_comment":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/7225/commits"},"statuses":{"href":"https://api.github.com/repos/rg3/youtube-dl/statuses/b243340f0ce311443a15a2dfd4356a9504e18c04"}}}}