Review of Notes indentation

This patch fix : https://github.com/StevenBlack/hosts/pull/520#discussion_r171971481
+ It also fix (forgoten coma) : https://github.com/StevenBlack/hosts/pull/520#discussion_r171971574
This commit is contained in:
funilrys 2018-03-02 22:53:15 +01:00
parent bebf7744ca
commit 6e62383b28
No known key found for this signature in database
GPG Key ID: 0D8BFEF5515C00C6

View File

@ -1147,12 +1147,12 @@ def domain_to_idna(line):
Notes
-----
- This function encode only the domain to `idna` format because in
most cases the encoding issue is due to a domain which looks like
most cases, the encoding issue is due to a domain which looks like
`b'\xc9\xa2oogle.com'.decode('idna')`.
- About the splitting:
We split because we only want to encode the domain and not the full
line which may cause some issue. Keep in mind that we split but we
still concatenate once we encoded the domain.
line which may cause some issue. Keep in mind that we split but we
still concatenate once we encoded the domain.
- The following split the prefix `0.0.0.0` or `127.0.0.1` of a line.
- The following also split the trailing comment of a given line.