Remove cloned branches

.clang-tidy

@@ -4,7 +4,6 @@ Checks: >
   -*-magic-numbers,
   -*-narrowing-conversions,
   -*-uppercase-literal-suffix,
-  -bugprone-branch-clone,
   -bugprone-suspicious-string-compare,
   -cert-err34-c,
   -clang-analyzer-unix.Malloc,

src/sratom.c

@@ -554,7 +554,7 @@ read_resource(Sratom*         sratom,
 static uint32_t
 atom_size(Sratom* sratom, uint32_t type_urid)
 {
-	if (type_urid == sratom->forge.Int) {
+	if (type_urid == sratom->forge.Int || type_urid == sratom->forge.Bool) {
 		return sizeof(int32_t);
 	} else if (type_urid == sratom->forge.Long) {
 		return sizeof(int64_t);
@@ -562,8 +562,6 @@ atom_size(Sratom* sratom, uint32_t type_urid)
 		return sizeof(float);
 	} else if (type_urid == sratom->forge.Double) {
 		return sizeof(double);
-	} else if (type_urid == sratom->forge.Bool) {
-		return sizeof(int32_t);
 	} else if (type_urid == sratom->forge.URID) {
 		return sizeof(uint32_t);
 	}

tests/sratom_test.c

@@ -383,13 +383,10 @@ test(SerdEnv* env, bool top_level, bool pretty_numbers)
 static int
 test_env(SerdEnv* env)
 {
-	if (test(env, false, false)) {
-		return 1;
-	} else if (test(env, true, false)) {
-		return 1;
-	} else if (test(env, false, true)) {
-		return 1;
-	} else if (test(env, true, true)) {
+	if (test(env, false, false) || //
+	    test(env, true, false) ||  //
+	    test(env, false, true) ||  //
+	    test(env, true, true)) {
 		return 1;
 	}