It works.. I guess? 🤣🤣

why is it type long long tho?

Half the conditions are redundant, but what would you expect from that code...

Everything is four anyway

Why don't you just convert it to a string and return the number of chars? 🤔

@They shhh

@They because that is ~80 times more expensive. Log10 is an infinitely better choice.

@calmyourtities because what other type can fit that much information?!

Benchmark this versus all other approaches. You'll find this (or rather: an optimized version of this killing half the compares) is the most performant.

When every cycle matters, all the fancy approaches (especially anything using friggin' string casting) are worse than useless.

Come on. It's like people don't understand memory and clocks even exist. 😡

@Root I almost thought that noone would understand it in the age of VM languages or JavaScript / python all over the place... Luv ya'

It’s called art.

@Root in that case I would expect a comment explaining exactly that. Even if it is faster, you need to stop people from being "smart" trying to refactor it.

Such hacks should be documented if they're needed.

@Root it is only going to return a number less than 255, why not make it a short

@calmyourtities ahh I assumed you meant the arg type; apologies! Yeah. No idea. It should absolutely be a byte/unsigned char/short, though.

@calmyourtities @Root not at all. You should use the types defines in stdint.h. Everything else is dangerous and should be avoided.