]> git.lizzy.rs Git - rust.git/commitdiff
Rollup merge of #74409 - LukasKalbertodt:improve-debug-impl-of-socketaddr-ipaddr...
authorTyler Mandry <tmandry@gmail.com>
Fri, 14 Aug 2020 21:46:44 +0000 (14:46 -0700)
committerGitHub <noreply@github.com>
Fri, 14 Aug 2020 21:46:44 +0000 (14:46 -0700)
Change Debug impl of SocketAddr and IpAddr to match their Display output

This has already been done for `SocketAddrV4`, `SocketAddrV6`, `IpAddrV4` and `IpAddrV6`. I don't see a point to keep the rather bad to read derived impl, especially so when pretty printing:

    V4(
        127.0.0.1
    )

From the `Display`, one can easily and unambiguously see if it's V4 or V6. Two examples:

```
127.0.0.1:443
[2001:db8:85a3::8a2e:370:7334]:443
```

Luckily the docs explicitly state that `Debug` output is not stable and that it may be changed at any time.

Using `Display` as `Debug` is very convenient for configuration structs (e.g. for webservers) that often just have a `derive(Debug)` and are printed that way to the one starting the server.

1  2 
library/std/src/net/ip.rs

Simple merge