[[ Bug 14645 ]] Check for errors in MCField::formattabalignments

livecodeian · livecodeian · commit 7213407882b2 · 2016-09-23T12:12:13.000+01:00
[[ Bug 14645 ]] Fix stack name in styledtext test file
diff --git a/engine/src/field.cpp b/engine/src/field.cpp
@@ -1550,26 +1550,27 @@ bool MCField::formattabalignments(const intenum_t *p_alignments, uindex_t p_alig
         switch (p_alignments[i])
         {
             case kMCParagraphTextAlignLeft:
-                /* UNCHECKED */ MCListAppendCString(*t_list, "left");
+                if (!MCListAppendCString(*t_list, "left"))
+					return false;
                 break;
                 
             case kMCParagraphTextAlignRight:
-                /* UNCHECKED */ MCListAppendCString(*t_list, "right");
+                if (!MCListAppendCString(*t_list, "right"))
+					return false;
                 break;
                 
             case kMCParagraphTextAlignCenter:
-                /* UNCHECKED */ MCListAppendCString(*t_list, "center");
+                if (!MCListAppendCString(*t_list, "center"))
+					return false;
                 break;
                 
             case kMCParagraphTextAlignJustify:
-                /* UNCHECKED */ MCListAppendCString(*t_list, "justify");
+                if (!MCListAppendCString(*t_list, "justify"))
+					return false;
                 break;
                 
             default:
-            {
-                MCAssert(false);
-                return false;
-            }
+				MCUnreachableReturn(false);
         }
     }
     
diff --git a/tests/lcs/core/field/styledText.livecodescript b/tests/lcs/core/field/styledText.livecodescript
@@ -1,4 +1,4 @@
-script "CoreFieldHtmlText"
+script "CoreFieldStyledText"
 /*
 Copyright (C) 2016 LiveCode Ltd.
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-script "CoreFieldHtmlText"`
	`1`	`+script "CoreFieldStyledText"`
`2`	`2`	`/*`
`3`	`3`	`Copyright (C) 2016 LiveCode Ltd.`
`4`	`4`