From beadb783a16f73873e4c6129b091ad86c1f9a53e Mon Sep 17 00:00:00 2001
From: Arjo Chakravarty <arjoc@intrinsic.ai>
Date: Thu, 12 Sep 2024 16:19:04 -0700
Subject: [PATCH] Send empty string instead of using sentinel value.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
---
 include/gz/sim/components/Model.hh | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/include/gz/sim/components/Model.hh b/include/gz/sim/components/Model.hh
index ed9967698a..f2f2ebfceb 100644
--- a/include/gz/sim/components/Model.hh
+++ b/include/gz/sim/components/Model.hh
@@ -75,8 +75,7 @@ namespace serializers
         }
       }
 
-      // Why bother even sending this. Wouldn't a blank string work?
-      if (skip)
+      if (!skip)
       {
       _out << "<?xml version=\"1.0\" ?>"
            << "<sdf version='" << SDF_PROTOCOL_VERSION << "'>"