Alex / KATT

Results 435 comments of Alex / KATT

Seemed like a majority was in favour, do a pull request?

They separated the condition and action on multiple lines, it wouldn't happen using the above example. Their bug: ``` c if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0) goto fail; //...

Maybe the condition is too edgy to be in the style guides. Opinions?

I'm :thumbsdown: to `switch`es in general.

Maybe have guideline of not using `switch` instead?

How about a simple `_`-prefix? That's how it's usually approached in JS.

Ah yeah, I thought that was the case, but didn't bother to google it. :)

How about `__`?

Otherwise, +1 on `p_`.