From 6540f16da1fc03f18f0e382ba8027db6791a9a75 Mon Sep 17 00:00:00 2001
From: ccpixel <cecil@posteo.net>
Date: Sun, 17 Nov 2024 09:40:49 -0500
Subject: [PATCH] Apply suggestions from code review
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
---
 scene/resources/packed_scene.cpp | 4 ++--
 tests/scene/test_packed_scene.h  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/scene/resources/packed_scene.cpp b/scene/resources/packed_scene.cpp
index b81c1eb2831f..2b3f37a732c2 100644
--- a/scene/resources/packed_scene.cpp
+++ b/scene/resources/packed_scene.cpp
@@ -1056,13 +1056,13 @@ Error SceneState::_parse_connections(Node *p_owner, Node *p_node, HashMap<String
 			const Node::Connection &c = F;
 
 			// Don't save connections that are not persistent.
-			if (!bool(c.flags & CONNECT_PERSIST)) {
+			if (!(c.flags & CONNECT_PERSIST)) {
 				continue;
 			}
 
 			// Don't include signals that are from scene instances
 			// (they are already saved in the scenes themselves).
-			if (bool(c.flags & CONNECT_INHERITED)) {
+			if (c.flags & CONNECT_INHERITED) {
 				continue;
 			}
 
diff --git a/tests/scene/test_packed_scene.h b/tests/scene/test_packed_scene.h
index 867495ebcb45..3698b00c6b86 100644
--- a/tests/scene/test_packed_scene.h
+++ b/tests/scene/test_packed_scene.h
@@ -56,7 +56,7 @@ TEST_CASE("[PackedScene] Pack Scene and Retrieve State") {
 	memdelete(scene);
 }
 
-TEST_CASE("[PackedScene] Test That Correct Signals are Preserved when Packing Scene") {
+TEST_CASE("[PackedScene] Signals Preserved when Packing Scene") {
 	// Create main scene
 	// root
 	// `- sub_node (local)