r20803 by jghali -

scribus-commit scribus-commit at lists.scribus.net
Sat Jan 30 21:17:40 UTC 2016


Author: jghali
Date: Sat Jan 30 21:17:40 2016
New Revision: 20803

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=20803
Log:
coverity: "array compared against 0" errors in Cpalette

Modified:
    trunk/Scribus/scribus/ui/cpalette.cpp

Modified: trunk/Scribus/scribus/ui/cpalette.cpp
URL: http://scribus.net/websvn/diff.php?repname=Scribus&rev=20803&path=/trunk/Scribus/scribus/ui/cpalette.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/cpalette.cpp	(original)
+++ trunk/Scribus/scribus/ui/cpalette.cpp	Sat Jan 30 21:17:40 2016
@@ -1367,10 +1367,10 @@
 		if(UndoManager::undoEnabled())
 		{
 			ss = new ScItemState<QPair<meshPoint,meshPoint> >(Um::GradPos);
-			ss->set("MOVE_MESH_PATCH","move_mesh_patch");
-			ss->set("ARRAY",false);
-			ss->set("X",grow);
-			ss->set("Y",gcol);
+			ss->set("MOVE_MESH_PATCH");
+			ss->set("ARRAY", false);
+			ss->set("X", grow);
+			ss->set("Y", gcol);
 		}
 		switch (gcol)
 		{
@@ -1436,10 +1436,10 @@
 		if(UndoManager::undoEnabled())
 		{
 			ScItemState<QPair<meshPoint,meshPoint> > *ss = new ScItemState<QPair<meshPoint,meshPoint> >(Um::GradPos);
-			ss->set("MOVE_MESH_PATCH","move_mesh_patch");
-			ss->set("ARRAY",true);
-			ss->set("X",grow);
-			ss->set("Y",gcol);
+			ss->set("MOVE_MESH_PATCH");
+			ss->set("ARRAY", true);
+			ss->set("X", grow);
+			ss->set("Y", gcol);
 			ss->setItem(qMakePair(currentItem->meshGradientArray[grow][gcol],tmp));
 			undoManager->action(currentItem,ss);
 		}
@@ -1462,10 +1462,10 @@
 		if(UndoManager::undoEnabled())
 		{
 			ss = new ScItemState<QPair<meshPoint,meshPoint> >(Um::GradPos);
-			ss->set("MOVE_MESH_PATCH","move_mesh_patch");
-			ss->set("ARRAY",false);
-			ss->set("X",grow);
-			ss->set("Y",gcol);
+			ss->set("MOVE_MESH_PATCH");
+			ss->set("ARRAY", false);
+			ss->set("X", grow);
+			ss->set("Y", gcol);
 		}
 		switch (gcol)
 		{
@@ -1525,10 +1525,10 @@
 		if(UndoManager::undoEnabled())
 		{
 			ScItemState<QPair<meshPoint,meshPoint> > *ss = new ScItemState<QPair<meshPoint,meshPoint> >(Um::GradPos);
-			ss->set("MOVE_MESH_PATCH","move_mesh_patch");
-			ss->set("ARRAY",true);
-			ss->set("X",grow);
-			ss->set("Y",gcol);
+			ss->set("MOVE_MESH_PATCH");
+			ss->set("ARRAY", true);
+			ss->set("X", grow);
+			ss->set("Y", gcol);
 			ss->setItem(qMakePair(currentItem->meshGradientArray[grow][gcol],tmp));
 			undoManager->action(currentItem,ss);
 		}
@@ -1686,9 +1686,9 @@
 		if(UndoManager::undoEnabled())
 		{
 			ScItemState<meshGradientPatch> *ss = new ScItemState<meshGradientPatch>(Um::RemoveMeshPatch,"",Um::ILine);
-			ss->set("REMOVE_MESH_PATCH","remove_mesh_patch");
+			ss->set("REMOVE_MESH_PATCH");
 			ss->setItem(currentItem->meshGradientPatches.takeAt(currentItem->selectedMeshPointX));
-			ss->set("POS",currentItem->selectedMeshPointX);
+			ss->set("POS", currentItem->selectedMeshPointX);
 			undoManager->action(currentItem,ss);
 		}
 		currentItem->selectedMeshPointX = -1;




More information about the scribus-commit mailing list