R-yan/l1ving_youtube-dl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Call _hook_progress() instead of faking it

Browse Source 
Since now report_progress() handles lack of a file size.
This commit is contained in:
John Hawkinson 2016-10-09 10:38:00 -04:00
parent 5a1e040b8e
commit f1f0b34a4e
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
youtube_dl/downloader/external.py
View File

@ -28,9 +28,7 @@ class ExternalFD(FileDownloader):
retval = self._call_downloader(tmpfilename, info_dict) retval = self._call_downloader(tmpfilename, info_dict)
if retval == 0: if retval == 0:
if filename == '-': if filename == '-':
# xxx report_progress() expects total_bytes to be set or it throws a KeyError, so self._hook_progress({'status': 'finished'})
# we can't just call: self._hook_progress({'status': 'finished'})
self.to_screen('[download] Download completed')
else: else:
fsize = os.path.getsize(encodeFilename(tmpfilename)) fsize = os.path.getsize(encodeFilename(tmpfilename))
self.to_screen('\r[%s] Downloaded %s bytes' % (self.get_basename(), fsize)) self.to_screen('\r[%s] Downloaded %s bytes' % (self.get_basename(), fsize))