[NEW] use native `bool` type in Valkey about valkey HOT 7 OPEN

The only thing I don't like about bool is that:

int isTrue() { return 1 }

is not the same as

bool isTrue() { return true}

Since a bool is only 1 byte. We have actually had production outages because of that mistake at AWS, because we internally use bool in various places, and people didn't understand the difference.

More generally, I prefer the code to more or less all use the same conventions. It's much more important to me to be consistent, since the more we diverge the more folks will poke in and say they want to do something else because that is the way that they like it.

from valkey.

What's a c++ comment?

I've never heard these be called C++ comments, always single line comments. That just might be because I never write C++ code though :D

from valkey.

What's a c++ comment?

from valkey.

What's a c++ comment?

// "C++ style" line comments are valid in C99

from valkey.

Let's keep these conventions then. Are they actually written down anywhere?

from valkey.

They're coming from c++ originally i believe.

https://en.cppreference.com/w/c/comment

from valkey.

I remember in very old C, there is no bool type, it is not like C++, which has built-in data type boolean.
After C99, C import bool type, but in low level, i think it need to be converted into integer zero or non-zero value.
Thus, i think in valkey, we do not need to import #include <stdbool.h>.

Personally, I have C++ experience. I still remember when I read redis code initially, because Redis is written by pure C language, I am confused by some C syntax.

from valkey.