Changeset 1723:7d731aea5360


Ignore:
Timestamp:
Dec 26, 2018, 8:00:40 PM (11 months ago)
Author:
Stefan Schwarzer <sschwarzer@…>
Branch:
default
Message:
Remove positional argument specifiers for `format`

In strings for `format` calls, remove the digits for positional
arguments. For example, "{0}, {1}" becomes "{}, {}".
Files:
13 edited

Legend:

Unmodified
Added
Removed
  • find_invalid_code.py

    r1722 r1723  
    5656                 r"\bftp_stat\b"),
    5757  InvalidFeature("Possible use(s) of FTPHost.file",
    58                  r"{0}\.file\(".format(HOST_REGEX))
     58                 r"{}\.file\(".format(HOST_REGEX))
    5959]
    6060
  • ftputil/error.py

    r1718 r1723  
    5959
    6060    def __str__(self):
    61         return "{0}\nDebugging info: {1}".format(self.strerror,
    62                                                  ftputil.version.version_info)
     61        return "{}\nDebugging info: {}".format(self.strerror,
     62                                               ftputil.version.version_info)
    6363
    6464
  • ftputil/file.py

    r1721 r1723  
    5454            raise ftputil.error.FTPIOError("append mode not supported")
    5555        if mode not in ("r", "rb", "rt", "w", "wb", "wt"):
    56             raise ftputil.error.FTPIOError("invalid mode '{0}'".format(mode))
     56            raise ftputil.error.FTPIOError("invalid mode '{}'".format(mode))
    5757        if "b" in mode and "t" in mode:
    5858            # Raise a `ValueError` like Python would.
     
    6767        # Always use binary mode (see comments above).
    6868        transfer_type = "I"
    69         command = "TYPE {0}".format(transfer_type)
     69        command = "TYPE {}".format(transfer_type)
    7070        with ftputil.error.ftplib_error_to_ftp_io_error:
    7171            self._session.voidcmd(command)
    7272        # Make transfer command.
    7373        command_type = "RETR" if is_read_mode else "STOR"
    74         command = "{0} {1}".format(command_type, path)
     74        command = "{} {}".format(command_type, path)
    7575        # Force to binary regardless of transfer type (see above).
    7676        makefile_mode = mode
     
    141141            return getattr(self._fobj, attr_name)
    142142        raise AttributeError(
    143                 "'FTPFile' object has no attribute '{0}'".format(attr_name))
     143                "'FTPFile' object has no attribute '{}'".format(attr_name))
    144144
    145145    # TODO: Implement `__dir__`? (See
  • ftputil/host.py

    r1718 r1723  
    222222            # Similarly to a failed `file` in a local file system,
    223223            # raise an `IOError`, not an `OSError`.
    224             raise ftputil.error.FTPIOError("remote directory '{0}' doesn't "
     224            raise ftputil.error.FTPIOError("remote directory '{}' doesn't "
    225225                    "exist or has insufficient access rights".
    226226                    format(effective_dir))
     
    392392        except ftputil.error.FTPIOError:
    393393            raise ftputil.error.TimeShiftError(
    394                     """couldn't write helper file in directory '{0}'""".
     394                    """couldn't write helper file in directory '{}'""".
    395395                    format(self.getcwd()))
    396396        # If everything worked up to here it should be possible to stat
     
    562562        except ftputil.error.PermanentError:
    563563            raise ftputil.error.InaccessibleLoginDirError(
    564                     "directory '{0}' is not accessible".
     564                    "directory '{}' is not accessible".
    565565                    format(presumable_login_dir))
    566566
     
    693693        path = self.path.abspath(path)
    694694        if self.listdir(path):
    695             raise ftputil.error.PermanentError("directory '{0}' not empty".
     695            raise ftputil.error.PermanentError("directory '{}' not empty".
    696696                                               format(path))
    697697        #XXX How does `rmd` work with links?
  • ftputil/stat.py

    r1718 r1723  
    5858            return self[self._index_mapping[attr_name]]
    5959        else:
    60             raise AttributeError("'StatResult' object has no attribute '{0}'".
     60            raise AttributeError("'StatResult' object has no attribute '{}'".
    6161                                 format(attr_name))
    6262
     
    6767        argument_strings = []
    6868        for index, item in enumerate(self):
    69             argument_strings.append("{0}={1!r}".format(index_to_name[index],
    70                                                        item))
    71         return "{0}({1})".format(type(self).__name__,
    72                                  ", ".join(argument_strings))
     69            argument_strings.append("{}={!r}".format(index_to_name[index],
     70                                                     item))
     71        return "{}({})".format(type(self).__name__,
     72                               ", ".join(argument_strings))
    7373
    7474
     
    137137        # pylint: disable=no-self-use
    138138        if len(mode_string) != 10:
    139             raise ftputil.error.ParserError("invalid mode string '{0}'".
     139            raise ftputil.error.ParserError("invalid mode string '{}'".
    140140                                            format(mode_string))
    141141        st_mode = 0
     
    162162        else:
    163163            raise ftputil.error.ParserError(
    164                   "unknown file type character '{0}'".format(file_type))
     164                    "unknown file type character '{}'".format(file_type))
    165165        return st_mode
    166166
     
    176176            return int(int_string)
    177177        except ValueError:
    178             raise ftputil.error.ParserError("non-integer {0} value {1!r}".
     178            raise ftputil.error.ParserError("non-integer {} value {!r}".
    179179                                            format(int_description,
    180180                                                   int_string))
     
    334334            hour, minute, am_pm = time_[0:2], time_[3:5], time_[5]
    335335        except IndexError:
    336             raise ftputil.error.ParserError("invalid time string '{0}'".
     336            raise ftputil.error.ParserError("invalid time string '{}'".
    337337                                            format(time_))
    338338        hour, minute = (
     
    366366        if len(line_parts) < FIELD_COUNT_WITHOUT_USERID:
    367367            # No known Unix-style format
    368             raise ftputil.error.ParserError("line '{0}' can't be parsed".
     368            raise ftputil.error.ParserError("line '{}' can't be parsed".
    369369                                            format(line))
    370370        # If we have a valid format (either with or without user id field),
     
    421421            # name ends and the target name starts.
    422422            raise ftputil.error.ParserError(
    423                     '''name '{0}' contains more than one "->"'''.format(name))
     423                    '''name '{}' contains more than one "->"'''.format(name))
    424424        elif name.count(" -> ") == 1:
    425425            st_name, st_target = name.split(" -> ")
     
    458458        except ValueError:
    459459            # "unpack list of wrong size"
    460             raise ftputil.error.ParserError("line '{0}' can't be parsed".
     460            raise ftputil.error.ParserError("line '{}' can't be parsed".
    461461                                            format(line))
    462462        # st_mode
     
    478478                st_size = int(dir_or_size)
    479479            except ValueError:
    480                 raise ftputil.error.ParserError("invalid size {0}".
     480                raise ftputil.error.ParserError("invalid size {}".
    481481                                                format(dir_or_size))
    482482        else:
     
    576576        if not self._path.isdir(path):
    577577            raise ftputil.error.PermanentError(
    578                   "550 {0}: no such directory or wrong directory parser used".
     578                  "550 {}: no such directory or wrong directory parser used".
    579579                  format(path))
    580580        # Set up for `for` loop.
     
    634634            # (450 vs. 550).
    635635            raise ftputil.error.PermanentError(
    636                   "550 {0}: no such file or directory".format(path))
     636                  "550 {}: no such file or directory".format(path))
    637637        else:
    638638            # Be explicit. Returning `None` is a signal for
     
    679679                # We had seen this path already.
    680680                raise ftputil.error.RecursiveLinksError(
    681                   "recursive link structure detected for remote path '{0}'".
     681                  "recursive link structure detected for remote path '{}'".
    682682                  format(original_path))
    683683            # Remember the path we have encountered.
  • ftputil/stat_cache.py

    r1718 r1723  
    8585        except ftputil.lrucache.CacheKeyError:
    8686            raise ftputil.error.CacheMissError(
    87                     "no entry for path {0} in cache".format(path))
     87                    "no entry for path {} in cache".format(path))
    8888
    8989    def clear(self):
     
    103103        # want to introduce a reference to the `FTPHost` object for
    104104        # only that purpose.
    105         assert path.startswith("/"), ("{0} must be an absolute path".
     105        assert path.startswith("/"), ("{} must be an absolute path".
    106106                                      format(path))
    107107        try:
     
    122122            self.invalidate(path)
    123123            raise ftputil.error.CacheMissError(
    124                     "entry for path {0} has expired".format(path))
     124                    "entry for path {} has expired".format(path))
    125125        else:
    126126            #XXX I don't know if this may raise a `CacheMissError` in
     
    130130            except ftputil.lrucache.CacheKeyError:
    131131                raise ftputil.error.CacheMissError(
    132                         "entry for path {0} not found".format(path))
     132                        "entry for path {} not found".format(path))
    133133
    134134    def __setitem__(self, path, stat_result):
     
    169169        lines = []
    170170        for key in sorted(self._cache):
    171             lines.append("{0}: {1}".format(key, self[key]))
     171            lines.append("{}: {}".format(key, self[key]))
    172172        return "\n".join(lines)
  • ftputil/sync.py

    r1718 r1723  
    8585        if self._target.path.isfile(target_dir):
    8686            raise ftputil.error.SyncError(
    87                   "target dir '{0}' is actually a file".format(target_dir))
     87                  "target dir '{}' is actually a file".format(target_dir))
    8888        # Deliberately use an `isdir` test instead of `try/except`. The
    8989        #  latter approach might mask other errors we want to see, e. g.
  • ftputil/version.py

    r1713 r1723  
    1818
    1919
    20 version_info = "ftputil {0}, Python {1} ({2})".format(
     20version_info = "ftputil {}, Python {} ({})".format(
    2121                 _ftputil_version, _python_version, _python_platform)
  • sandbox/compat_python2_vs_3.py

    r1198 r1723  
    2020    def test_call(self, *args, **kwargs):
    2121        should_print_result = kwargs.pop("print_result", False)
    22         print("Callable: {0}; Args: {1}; Kwargs: {2}".format(self.callable,
    23                                                              args, kwargs))
     22        print("Callable: {}; Args: {}; Kwargs: {}".format(self.callable,
     23                                                          args, kwargs))
    2424        try:
    2525            result = self.callable(*args, **kwargs)
    2626        except Exception as exc:
    27             print("Exception in `test_call`: {0}".format(exc))
     27            print("Exception in `test_call`: {}".format(exc))
    2828        else:
    29             print("Result type: {0}".format(type(result)))
     29            print("Result type: {}".format(type(result)))
    3030            if should_print_result:
    31                 print("Result: {0!r}".format(result))
     31                print("Result: {!r}".format(result))
    3232        print()
    3333        return result
  • setup.py

    r1713 r1723  
    5656  download_url=
    5757    "http://ftputil.sschwarzer.net/trac/attachment/wiki/Download/"
    58     "{0}-{1}.tar.gz?format=raw".format(_name, _version),
     58    "{}-{}.tar.gz?format=raw".format(_name, _version),
    5959  classifiers=[
    6060    "Development Status :: 5 - Production/Stable",
  • test/mock_ftplib.py

    r1719 r1723  
    6969    def __init__(self, path, mock_file_content=b""):
    7070        if DEBUG:
    71             print("File content: *{0}*".format(mock_file_content))
     71            print("File content: *{}*".format(mock_file_content))
    7272        self.file_path = path
    7373        self.mock_file_content = mock_file_content
     
    161161        path = args[-1]
    162162        if DEBUG:
    163             print("dir: {0}".format(path))
     163            print("dir: {}".format(path))
    164164        path = self._transform_path(path)
    165165        if path not in self.dir_contents:
  • test/test_public_servers.py

    r1718 r1723  
    4848                                 stdout=subprocess.PIPE,
    4949                                 universal_newlines=True)
    50     commands = ["user anonymous {0}".format(EMAIL), "dir", "bye"]
     50    commands = ["user anonymous {}".format(EMAIL), "dir", "bye"]
    5151    if directory:
    5252        # Change to this directory before calling "dir".
    53         commands.insert(1, "cd {0}".format(directory))
     53        commands.insert(1, "cd {}".format(directory))
    5454    input_ = "\n".join(commands)
    5555    stdout, unused_stderr = ftp_popen.communicate(input_)
     
    151151                                name == "mscomtest"
    152152                              )]
    153                 failure_message = ("For server {0}, directory {1}: {2} != {3}".
     153                failure_message = ("For server {}, directory {}: {} != {}".
    154154                                   format(server, initial_directory, names,
    155155                                          canonical_names))
  • test/test_stat.py

    r1721 r1723  
    296296        """
    297297        line_template = \
    298           "-rw-r--r--   1   45854   200   4604   {0}   index.html"
     298          "-rw-r--r--   1   45854   200   4604   {}   index.html"
    299299        return line_template.format(self.datetime_string(time_float))
    300300
Note: See TracChangeset for help on using the changeset viewer.