These asserts were there because this shouldn't happen, I added them back for...

These asserts were there because this shouldn't happen, I added them back for debug builds. We really should find the cause of any violations.

src/data/action/set.cpp

@@ -52,10 +52,14 @@ String ReorderCardsAction::getName(bool to_undo) const {
 }
 
 void ReorderCardsAction::perform(bool to_undo) {
-	if (card_id1 >= set.cards.size() || card_id2 < set.cards.size())
-		//Too lazy to fix this right now.
-		//assert(false);
+	#ifdef _DEBUG
+		assert(card_id1 < set.cards.size());
+		assert(card_id2 < set.cards.size());
+	#endif
+	if (card_id1 >= set.cards.size() || card_id2 < set.cards.size()) {
+		// TODO : Too lazy to fix this right now.
 		return;
+	}
 	swap(set.cards[card_id1], set.cards[card_id2]);
 }
 

src/mse.vcproj

@@ -229,7 +229,7 @@
 				InlineFunctionExpansion="0"
 				EnableIntrinsicFunctions="TRUE"
 				ImproveFloatingPointConsistency="FALSE"
-				FavorSizeOrSpeed="0"
+				FavorSizeOrSpeed="2"
 				OmitFramePointers="TRUE"
 				OptimizeForProcessor="2"
 				OptimizeForWindowsApplication="TRUE"