r20898 by craig -

scribus-commit scribus-commit at lists.scribus.net
Thu Feb 4 22:11:25 UTC 2016


Author: craig
Date: Thu Feb  4 22:11:25 2016
New Revision: 20898

URL: http://scribus.net/websvn/listing.php?repname=Scribus&sc=1&rev=20898
Log:
coverity #1350103: See if this fixes this control flow issue due to lack of brackets, incorrect operator precedence

Modified:
    trunk/Scribus/scribus/appmodehelper.cpp

Modified: trunk/Scribus/scribus/appmodehelper.cpp
URL: http://scribus.net/websvn/diff.php?repname=Scribus&rev=20898&path=/trunk/Scribus/scribus/appmodehelper.cpp
==============================================================================
--- trunk/Scribus/scribus/appmodehelper.cpp	(original)
+++ trunk/Scribus/scribus/appmodehelper.cpp	Thu Feb  4 22:11:25 2016
@@ -1277,7 +1277,7 @@
 	// Determine state.
 	PageItem* item = doc ? doc->m_Selection->itemAt(0) : 0;
 	PageItem_Table* table = (item && item->isTable()) ? item->asTable() : 0;
-	const bool tableEdit = table && doc->appMode == modeEditTable;
+	const bool tableEdit = table && (doc->appMode == modeEditTable);
 	const int tableRows = table ? table->rows() : 0;
 	const int tableColumns = table ? table->columns() : 0;
 	const int selectedRows = tableEdit ? table->selectedRows().size() : 0;




More information about the scribus-commit mailing list