[royale-asjs] branch develop updated: Changed errors to asserts

[harbs]

This is an automated email from the ASF dual-hosted git repository.

harbs pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new dedc8d2  Changed errors to asserts
     new 9b6bd0e  Merge branch 'develop' of 
https://github.com/apache/royale-asjs into develop
dedc8d2 is described below

commit dedc8d237a62bfe19598bb0370930f4b7971c960
Author: Harbs <ha...@in-tools.com>
AuthorDate: Sun Jan 3 21:30:01 2021 +0200

    Changed errors to asserts
---
 .../apache/royale/jewel/beads/layouts/HorizontalLayout.as  | 14 ++++++--------
 .../apache/royale/jewel/beads/layouts/VerticalLayout.as    | 14 ++++++--------
 2 files changed, 12 insertions(+), 16 deletions(-)

diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/HorizontalLayout.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/HorizontalLayout.as
index 15c515e..4b175af 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/HorizontalLayout.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/HorizontalLayout.as
@@ -29,6 +29,7 @@ package org.apache.royale.jewel.beads.layouts
        import org.apache.royale.core.ValuesManager;
        import org.apache.royale.core.layout.ILayoutStyleProperties;
        import org.apache.royale.events.Event;
+       import org.apache.royale.debugging.assert;
 
     /**
         *  The HorizontalLayout class is a horizontal layout
@@ -199,14 +200,11 @@ package org.apache.royale.jewel.beads.layouts
                COMPILE::JS
                private function setGap(value:Number):void
                {
-                       if (value >= 0 && value <= GapConstants.GAPS)
-                       {
-                               if (hostComponent.containsClass("gap-" + _gap + 
"x" + GapConstants.GAP_STEP + "px"))
-                                       hostComponent.removeClass("gap-" + _gap 
+ "x" + GapConstants.GAP_STEP + "px");
-                               if(value != 0)
-                                       hostComponent.addClass("gap-" + value + 
"x" + GapConstants.GAP_STEP + "px");
-                       } else
-                               throw new Error("Gap needs to be between 0 and 
" + GapConstants.GAPS + ", got " + value);
+                       assert(value >= 0 && value <= GapConstants.GAPS,"Gap 
needs to be between 0 and " + GapConstants.GAPS + ", got " + value);
+                       if (hostComponent.containsClass("gap-" + _gap + "x" + 
GapConstants.GAP_STEP + "px"))
+                               hostComponent.removeClass("gap-" + _gap + "x" + 
GapConstants.GAP_STEP + "px");
+                       if(value != 0)
+                               hostComponent.addClass("gap-" + value + "x" + 
GapConstants.GAP_STEP + "px");
                }
 
                private var variableRowHeightInitialized:Boolean;
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/VerticalLayout.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/VerticalLayout.as
index b907b81..d0d66b8 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/VerticalLayout.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/layouts/VerticalLayout.as
@@ -29,6 +29,7 @@ package org.apache.royale.jewel.beads.layouts
        import org.apache.royale.core.ValuesManager;
        import org.apache.royale.core.layout.ILayoutStyleProperties;
        import org.apache.royale.events.Event;
+       import org.apache.royale.debugging.assert;
 
        /**
         *  The VerticalLayout class is a vertical layout
@@ -199,14 +200,11 @@ package org.apache.royale.jewel.beads.layouts
                COMPILE::JS
                private function setGap(value:Number):void
                {
-                       if (value >= 0 && value <= GapConstants.GAPS)
-                       {
-                               if (hostComponent.containsClass("gap-" + _gap + 
"x" + GapConstants.GAP_STEP + "px"))
-                                       hostComponent.removeClass("gap-" + _gap 
+ "x" + GapConstants.GAP_STEP + "px");
-                               if(value != 0)
-                                       hostComponent.addClass("gap-" + value + 
"x" + GapConstants.GAP_STEP + "px");
-                       } else
-                               throw new Error("Gap needs to be between 0 and 
" + GapConstants.GAPS + ", got " + value);
+                       assert(value >= 0 && value <= GapConstants.GAPS,"Gap 
needs to be between 0 and " + GapConstants.GAPS + ", got " + value);
+                       if (hostComponent.containsClass("gap-" + _gap + "x" + 
GapConstants.GAP_STEP + "px"))
+                               hostComponent.removeClass("gap-" + _gap + "x" + 
GapConstants.GAP_STEP + "px");
+                       if(value != 0)
+                               hostComponent.addClass("gap-" + value + "x" + 
GapConstants.GAP_STEP + "px");
                }
 
                private var variableRowHeightInitialized:Boolean;