From 559dad93b882a35e1fc86fbc60149aec4391413f Mon Sep 17 00:00:00 2001
From: Robert Osfield <robert@openscenegraph.com>
Date: Thu, 11 Jan 2018 08:51:08 +0000
Subject: [PATCH] Added returns to avoid fall through between case statements

---
 src/osgSim/ShapeAttribute.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/osgSim/ShapeAttribute.cpp b/src/osgSim/ShapeAttribute.cpp
index c18f70cd8..753322d5e 100644
--- a/src/osgSim/ShapeAttribute.cpp
+++ b/src/osgSim/ShapeAttribute.cpp
@@ -135,11 +135,13 @@ int ShapeAttribute::compare(const osgSim::ShapeAttribute& sa) const
         {
             if (_string<sa._string) return -1;
             if (sa._string<_string) return 1;
+            return 0;
         }
         case DOUBLE:
         {
             if (_double<sa._double) return -1;
             if (sa._double<_double) return 1;
+            return 0;
         }
         case INTEGER:
         case UNKNOWN:
@@ -147,6 +149,7 @@ int ShapeAttribute::compare(const osgSim::ShapeAttribute& sa) const
         {
             if (_integer<sa._integer) return -1;
             if (sa._integer<_integer) return 1;
+            return 0;
         }
     }
     return 0;