Merge pull request #5426 from AnimatedRNG/geometry-check

Replaces DeveloperError(s) with Check(s) in Geometry
CesiumGS · Jun 6, 2017 · bb8989c · bb8989c
2 parents 7b5675e + f697b26
commit bb8989c
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/Source/Core/Geometry.js b/Source/Core/Geometry.js
@@ -1,11 +1,13 @@
 /*global define*/
 define([
+        './Check',
         './defaultValue',
         './defined',
         './DeveloperError',
         './GeometryType',
         './PrimitiveType'
     ], function(
+        Check,
         defaultValue,
         defined,
         DeveloperError,
@@ -67,9 +69,7 @@ define([
         options = defaultValue(options, defaultValue.EMPTY_OBJECT);
 
         //>>includeStart('debug', pragmas.debug);
-        if (!defined(options.attributes)) {
-            throw new DeveloperError('options.attributes is required.');
-        }
+        Check.typeOf.object('options.attributes', options.attributes);
         //>>includeEnd('debug');
 
         /**
@@ -173,9 +173,7 @@ define([
      */
     Geometry.computeNumberOfVertices = function(geometry) {
         //>>includeStart('debug', pragmas.debug);
-        if (!defined(geometry)) {
-            throw new DeveloperError('geometry is required.');
-        }
+        Check.typeOf.object('geometry', geometry);
         //>>includeEnd('debug');
 
         var numberOfVertices = -1;