svn commit: r447878 - in head/science/massxpert: . files

[Raphael Kubo da Costa]

Author: rakuco
Date: Sun Aug 13 11:19:52 2017
New Revision: 447878
URL: https://svnweb.freebsd.org/changeset/ports/447878

Log:
  Add patch to drop a wrong ';' in an if check.
  
  In addition to fixing an actual bug, this also fixes the build with GCC 6:
  
  /wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp: In member function 'bool massXpert::PropListHolder::removeProp(const QString&)':
  /wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp:216:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
    if(m_propList.at(iter)->name() == name);
    ^~
  /wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp:217:2: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
    {
  
  PR:		219293
  Approved by:	maintainer timeout (vg, 18 days)
  MFH:		2017Q3

Added:
  head/science/massxpert/files/patch-lib_propListHolder.cpp   (contents, props changed)
Modified:
  head/science/massxpert/Makefile

Modified: head/science/massxpert/Makefile
==============================================================================
--- head/science/massxpert/Makefile	Sun Aug 13 11:12:59 2017	(r447877)
+++ head/science/massxpert/Makefile	Sun Aug 13 11:19:52 2017	(r447878)
@@ -3,7 +3,7 @@
 
 PORTNAME=	massxpert
 PORTVERSION=	3.4.0
-PORTREVISION=	7
+PORTREVISION=	8
 CATEGORIES=	science
 MASTER_SITES=	http://download.tuxfamily.org/massxpert/source/
 

Added: head/science/massxpert/files/patch-lib_propListHolder.cpp
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/science/massxpert/files/patch-lib_propListHolder.cpp	Sun Aug 13 11:19:52 2017	(r447878)
@@ -0,0 +1,19 @@
+Fixes the build with GCC 6 (in addition to fixing an actual bug).
+
+/wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp: In member function 'bool massXpert::PropListHolder::removeProp(const QString&)':
+/wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp:216:2: error: this 'if' clause does not guard... [-Werror=misleading-indentation]
+  if(m_propList.at(iter)->name() == name);
+  ^~
+/wrkdirs/usr/ports/science/massxpert/work/massxpert-3.4.0/lib/propListHolder.cpp:217:2: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
+  {
+--- lib/propListHolder.cpp.orig	2017-07-26 17:06:18 UTC
++++ lib/propListHolder.cpp
+@@ -213,7 +213,7 @@ namespace massXpert
+   {
+     for (int iter = 0; iter < m_propList.size(); ++iter)
+       {
+-	if(m_propList.at(iter)->name() == name);
++	if(m_propList.at(iter)->name() == name)
+ 	{
+ 	  m_propList.removeAt(iter);
+