edg-l/ddnet
1
0
Fork 0
mirror of https://github.com/ddnet/ddnet.git synced 2024-11-17 13:38:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Document that using integers in boolean contexts is not recommended

Browse source
This commit is contained in:
heinrich5991 2024-11-15 22:46:00 +01:00
parent 666d9f06a9
commit 9ffde94c86
1 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
CONTRIBUTING.md
View file

@ -149,6 +149,22 @@ if(Foo) { .. }
Unless the alternative code is more complex and harder to read. Unless the alternative code is more complex and harder to read.
### Using integers in boolean contexts should be avoided
❌
```C++
int Foo = 0;
if(!Foo) { .. }
```
✅
```C++
int Foo = 0;
if(Foo != 0) { .. }
```
### Methods with default arguments should be avoided ### Methods with default arguments should be avoided
Default arguments tend to break quickly, if you have multiple you have to specify each even if you only want to change the last one. Default arguments tend to break quickly, if you have multiple you have to specify each even if you only want to change the last one.