Always break nested Conditionals

[belav]

When we have a nested conditional, we should probably break each one.

Currently it is like this.

return (h >= '0' && h <= '9')
  ? h - '0'
  : (h >= 'a' && h <= 'f') ? h - 'a' + 10 : (h >= 'A' && h <= 'F') ? h - 'A' + 10 : -1;

But I think it should look like this

return h >= "0" && h <= "9"
  ? h - "0"
  : h >= "a" && h <= "f"
      ? h - "a" + 10
      : h >= "A" && h <= "F"
          ? h - "A" + 10
          : -1;

Prettier does it like this, but I think the indentation makes it more readable

return h >= "0" && h <= "9"
  ? h - "0"
  : h >= "a" && h <= "f"
  ? h - "a" + 10
  : h >= "A" && h <= "F"
  ? h - "A" + 10
  : -1;

[respel]

I like your approach for this @belav. The prettier one is pretty(no pun intended :P) confusing.

I forgot to mention the one true correct way ™ to do it though. Just don't use ternary operators :)

[belav]

@respel you should look at some of the crazy nested ternary operators https://github.com/belav/csharpier-repos/pull/10/files