Merge pull request #462 from JordanMartinez/convertAssertToException

JordanMartinez · web-flow · commit 7bf8f4cae858 · 2017-03-22T22:19:17.000-07:00
Turn assert statement into IllegalArgumentException
diff --git a/richtextfx/src/main/java/org/fxmisc/richtext/model/Paragraph.java b/richtextfx/src/main/java/org/fxmisc/richtext/model/Paragraph.java
@@ -39,7 +39,9 @@ public Paragraph(PS paragraphStyle, SegmentOps<SEG, S> segmentOps, SEG text, SEG
     }
 
     Paragraph(PS paragraphStyle, SegmentOps<SEG, S> segmentOps, List<SEG> segments) {
-        assert !segments.isEmpty();
+        if (segments.isEmpty()) {
+            throw new IllegalArgumentException("Cannot construct a Paragraph with an empty list of segments");
+        }
 
         this.segmentOps = segmentOps;
         this.segments = segments;

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,9 @@ public Paragraph(PS paragraphStyle, SegmentOps<SEG, S> segmentOps, SEG text, SEG`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`Paragraph(PS paragraphStyle, SegmentOps<SEG, S> segmentOps, List<SEG> segments) {`
`42`		`- assert !segments.isEmpty();`
	`42`	`+ if (segments.isEmpty()) {`
	`43`	`+ throw new IllegalArgumentException("Cannot construct a Paragraph with an empty list of segments");`
	`44`	`+ }`
`43`	`45`
`44`	`46`	`this.segmentOps = segmentOps;`
`45`	`47`	`this.segments = segments;`