max is divided by 2

[tcharding]

EDIT: Probably not worth reading this issue, just see #71

I'm pretty sure this line of code is wrong but I can't come up with a unit test to prove it. Patching it to what I think it should be does not break any unit tests.

hex-conservative/src/display.rs

Line 169 in 7116a46

Some(max) if self.bytes.len() * 2 > (max + 1) / 2 => max,

We are dividing the max string length by 2 as well as multiplying the number of bytes by 2 - we should just do one (I believe).

            let string_len = match f.precision() {
                Some(max) if self.bytes.len() * 2 > (max + 1) / 2 => max,
                Some(_) | None => self.bytes.len() * 2,
            };

Should be I think

```rust
            let string_len = match f.precision() {
                Some(max) if self.bytes.len() * 2 > max => max,
                Some(_) | None => self.bytes.len() * 2,
            };

[tcharding]

ooo, (max + 1) / 2 might have been to handle odd length max width, which makes no sense for hex strings. So perhaps this should be

            let string_len = match f.precision() {
                // Do ceil division on max so as to ignore odd length
                // max width which makes no sense for hex strings.
                Some(max) if self.bytes.len() > (max + 1) / 2 => max,
                Some(_) | None => self.bytes.len() * 2,
            };

[tcharding]

Sorry for the noise, took me a while to work out the interaction between width and precision - TIL.