Use CODEPOINT_NONE instead of assigning negatives to unsigned

Monte Goulding · montegoulding · commit 0c867b7684ac · 2016-09-14T18:22:27.000+10:00
diff --git a/engine/src/block.cpp b/engine/src/block.cpp
@@ -756,12 +756,12 @@ bool MCBlock::fit(coord_t x, coord_t maxwidth, findex_t& r_break_index, bool& r_
 	if (t_next_block != parent -> getblocks())
 	{
 		if (t_next_block -> GetLength() == 0)
-			t_next_block_char = -2;
+			t_next_block_char = CODEPOINT_NONE-1;
 		else
 			t_next_block_char = parent->GetCodepointAtIndex(t_next_block -> m_index);
 	}
 	else
-		t_next_block_char = -1;
+		t_next_block_char = CODEPOINT_NONE;
 
     // FG-2013-10-21 [[ Field speedups ]]
     // Previously, we used to calculate the length of the entire block here in order
@@ -836,7 +836,7 @@ bool MCBlock::fit(coord_t x, coord_t maxwidth, findex_t& r_break_index, bool& r_
                     t_end_of_block = true;
                 }
                 
-                if (t_next_char == -1 ||
+                if (t_next_char == CODEPOINT_NONE ||
                     MCUnicodeCanBreakBetween(t_this_char, t_next_char))
                 {
                     t_can_break = true;

Original file line number	Diff line number	Diff line change
`@@ -756,12 +756,12 @@ bool MCBlock::fit(coord_t x, coord_t maxwidth, findex_t& r_break_index, bool& r_`
`756`	`756`	`if (t_next_block != parent -> getblocks())`
`757`	`757`	`{`
`758`	`758`	`if (t_next_block -> GetLength() == 0)`
`759`		`- t_next_block_char = -2;`
	`759`	`+ t_next_block_char = CODEPOINT_NONE-1;`
`760`	`760`	`else`
`761`	`761`	`t_next_block_char = parent->GetCodepointAtIndex(t_next_block -> m_index);`
`762`	`762`	`}`
`763`	`763`	`else`
`764`		`- t_next_block_char = -1;`
	`764`	`+ t_next_block_char = CODEPOINT_NONE;`
`765`	`765`
`766`	`766`	`// FG-2013-10-21 [[ Field speedups ]]`
`767`	`767`	`// Previously, we used to calculate the length of the entire block here in order`
`@@ -836,7 +836,7 @@ bool MCBlock::fit(coord_t x, coord_t maxwidth, findex_t& r_break_index, bool& r_`
`836`	`836`	`t_end_of_block = true;`
`837`	`837`	`}`
`838`	`838`
`839`		`- if (t_next_char == -1 \|\|`
	`839`	`+ if (t_next_char == CODEPOINT_NONE \|\|`
`840`	`840`	`MCUnicodeCanBreakBetween(t_this_char, t_next_char))`
`841`	`841`	`{`
`842`	`842`	`t_can_break = true;`