Re: [PATCH master] Pythonize some code from network.py for pleasure.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ack

--

  Martin Gracik

----- Original Message -----
> ---
> pyanaconda/network.py | 14 +++++++-------
> 1 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/pyanaconda/network.py b/pyanaconda/network.py
> index 36d8373..690069f 100644
> --- a/pyanaconda/network.py
> +++ b/pyanaconda/network.py
> @@ -71,7 +71,7 @@ class IPMissing(Exception):
> pass
> 
> def sanityCheckHostname(hostname):
> - if len(hostname) < 1:
> + if not hostname:
> return None
> 
> if len(hostname) > 255:
> @@ -80,12 +80,12 @@ def sanityCheckHostname(hostname):
> validStart = string.ascii_letters + string.digits
> validAll = validStart + ".-"
> 
> - if string.find(validStart, hostname[0]) == -1:
> + if hostname[0] not in validStart:
> return _("Hostname must start with a valid character in the ranges "
> "'a-z', 'A-Z', or '0-9'")
> 
> - for i in range(1, len(hostname)):
> - if string.find(validAll, hostname[i]) == -1:
> + for char in hostname[1:]:
> + if char not in validAll:
> return _("Hostnames can only contain the characters 'a-z', 'A-Z',
> '0-9', '-', or '.'")
> 
> return None
> @@ -129,13 +129,13 @@ def getDefaultHostname(anaconda):
> 
> # sanity check an IP string.
> def sanityCheckIPString(ip_string):
> - if ip_string.strip() == "":
> + if not ip_string.strip():
> raise IPMissing, _("IP address is missing.")
> 
> - if ip_string.find(':') == -1 and ip_string.find('.') > 0:
> + if '.' in ip_string[1:] and ':' not in ip_string:
> family = socket.AF_INET
> errstr = _("IPv4 addresses must contain four numbers between 0 and
> 255, separated by periods.")
> - elif ip_string.find(':') > 0 and ip_string.find('.') == -1:
> + elif ':' in ip_string[1:] and '.' not in ip_string:
> family = socket.AF_INET6
> errstr = _("'%s' is not a valid IPv6 address.") % ip_string
> else:
> --
> 1.7.4
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux