diff options
author | Florian Bruhin <git@the-compiler.org> | 2018-08-07 16:26:41 +0200 |
---|---|---|
committer | Florian Bruhin <git@the-compiler.org> | 2018-08-15 10:55:18 +0200 |
commit | bb8901f48f50a29d8bb7845879b356674427e805 (patch) | |
tree | c2be911dea69f613658ffddb6d43021e895d2fd2 | |
parent | cb76e78faf2b0f324efd4ed742949335df1d5aa5 (diff) | |
download | qutebrowser-bb8901f48f50a29d8bb7845879b356674427e805.tar.gz qutebrowser-bb8901f48f50a29d8bb7845879b356674427e805.zip |
Use exist_ok=True for os.path.makedirs
See #2570
(cherry picked from commit 1cb547a8de9228be81db912857cc0a61116d8c05)
-rw-r--r-- | qutebrowser/browser/downloads.py | 9 | ||||
-rw-r--r-- | qutebrowser/browser/urlmarks.py | 3 | ||||
-rw-r--r-- | qutebrowser/misc/lineparser.py | 3 | ||||
-rw-r--r-- | qutebrowser/utils/standarddir.py | 5 |
4 files changed, 5 insertions, 15 deletions
diff --git a/qutebrowser/browser/downloads.py b/qutebrowser/browser/downloads.py index 2e30c26c2..396f5152c 100644 --- a/qutebrowser/browser/downloads.py +++ b/qutebrowser/browser/downloads.py @@ -79,10 +79,7 @@ def download_dir(): else: ddir = directory - try: - os.makedirs(ddir) - except FileExistsError: - pass + os.makedirs(ddir, exist_ok=True) return ddir @@ -692,9 +689,7 @@ class AbstractDownloadItem(QObject): global last_used_directory try: - os.makedirs(os.path.dirname(self._filename)) - except FileExistsError: - pass + os.makedirs(os.path.dirname(self._filename), exist_ok=True) except OSError as e: self._die(e.strerror) diff --git a/qutebrowser/browser/urlmarks.py b/qutebrowser/browser/urlmarks.py index 0a0dfb4f2..bb20e2166 100644 --- a/qutebrowser/browser/urlmarks.py +++ b/qutebrowser/browser/urlmarks.py @@ -240,8 +240,7 @@ class BookmarkManager(UrlMarkManager): def _init_lineparser(self): bookmarks_directory = os.path.join(standarddir.config(), 'bookmarks') - if not os.path.isdir(bookmarks_directory): - os.makedirs(bookmarks_directory) + os.makedirs(bookmarks_directory, exist_ok=True) bookmarks_subdir = os.path.join('bookmarks', 'urls') self._lineparser = lineparser.LineParser( diff --git a/qutebrowser/misc/lineparser.py b/qutebrowser/misc/lineparser.py index 6e50edb9b..e9fc67e18 100644 --- a/qutebrowser/misc/lineparser.py +++ b/qutebrowser/misc/lineparser.py @@ -72,8 +72,7 @@ class BaseLineParser(QObject): Return: True if the file should be saved, False otherwise. """ - if not os.path.exists(self._configdir): - os.makedirs(self._configdir, 0o755) + os.makedirs(self._configdir, 0o755, exist_ok=True) return True def _after_save(self): diff --git a/qutebrowser/utils/standarddir.py b/qutebrowser/utils/standarddir.py index 40f1fa966..95aa4d438 100644 --- a/qutebrowser/utils/standarddir.py +++ b/qutebrowser/utils/standarddir.py @@ -290,10 +290,7 @@ def _create(path): 0700. If the destination directory exists already the permissions should not be changed. """ - try: - os.makedirs(path, 0o700) - except FileExistsError: - pass + os.makedirs(path, 0o700, exist_ok=True) def _init_dirs(args=None): |