From owner-svn-ports-head@FreeBSD.ORG Sun Mar 22 00:06:36 2015 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 87FCB8D4; Sun, 22 Mar 2015 00:06:36 +0000 (UTC) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:1900:2254:2068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 57A90DAD; Sun, 22 Mar 2015 00:06:36 +0000 (UTC) Received: from svn.freebsd.org ([127.0.1.70]) by svn.freebsd.org (8.14.9/8.14.9) with ESMTP id t2M06aRd024067; Sun, 22 Mar 2015 00:06:36 GMT (envelope-from marino@FreeBSD.org) Received: (from marino@localhost) by svn.freebsd.org (8.14.9/8.14.9/Submit) id t2M06ZdG024064; Sun, 22 Mar 2015 00:06:35 GMT (envelope-from marino@FreeBSD.org) Message-Id: <201503220006.t2M06ZdG024064@svn.freebsd.org> X-Authentication-Warning: svn.freebsd.org: marino set sender to marino@FreeBSD.org using -f From: John Marino Date: Sun, 22 Mar 2015 00:06:35 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r381882 - in head/science/massxpert: . files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 22 Mar 2015 00:06:36 -0000 Author: marino Date: Sun Mar 22 00:06:34 2015 New Revision: 381882 URL: https://svnweb.freebsd.org/changeset/ports/381882 QAT: https://qat.redports.org/buildarchive/r381882/ Log: science/massxpert: Fix two logical errors (found by gcc5) There were two boolean logic issues found by gcc5 when compiling massxpert. One was comparing boolean output to a negative integer (is always false) and the other applied the "not" operator in the wrong place. Fix provided per request by maintainer. PR: 197823 Added: head/science/massxpert/files/patch-lib_cleaveSpec.cpp (contents, props changed) head/science/massxpert/files/patch-plugins-src_seqToolsPlugin_seqToolsPluginDlg.cpp (contents, props changed) Modified: head/science/massxpert/Makefile Modified: head/science/massxpert/Makefile ============================================================================== --- head/science/massxpert/Makefile Sat Mar 21 23:46:17 2015 (r381881) +++ head/science/massxpert/Makefile Sun Mar 22 00:06:34 2015 (r381882) @@ -3,12 +3,12 @@ PORTNAME= massxpert PORTVERSION= 3.4.0 -PORTREVISION= 2 +PORTREVISION= 3 CATEGORIES= science MASTER_SITES= http://download.tuxfamily.org/massxpert/source/ MAINTAINER= vg@FreeBSD.org -COMMENT= The massXpert software for (bio)chemists +COMMENT= massXpert software for (bio)chemists LICENSE= GPLv3 Added: head/science/massxpert/files/patch-lib_cleaveSpec.cpp ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/science/massxpert/files/patch-lib_cleaveSpec.cpp Sun Mar 22 00:06:34 2015 (r381882) @@ -0,0 +1,11 @@ +--- lib/cleaveSpec.cpp.orig 2012-12-25 21:02:43 UTC ++++ lib/cleaveSpec.cpp +@@ -427,7 +427,7 @@ namespace massXpert + return false; + + +- if (parse() == -1) ++ if (parse() == false) + return false; + + // If there are rules, we have to check them all. Added: head/science/massxpert/files/patch-plugins-src_seqToolsPlugin_seqToolsPluginDlg.cpp ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/science/massxpert/files/patch-plugins-src_seqToolsPlugin_seqToolsPluginDlg.cpp Sun Mar 22 00:06:34 2015 (r381882) @@ -0,0 +1,11 @@ +--- plugins-src/seqToolsPlugin/seqToolsPluginDlg.cpp.orig 2012-12-25 21:02:43 UTC ++++ plugins-src/seqToolsPlugin/seqToolsPluginDlg.cpp +@@ -215,7 +215,7 @@ SeqToolsPluginDlg::loadDictionary () + + QStringList stringList = line.split (">", QString::SkipEmptyParts); + +- if (!stringList.size () == 2) ++ if (stringList.size () != 2) + { + QMessageBox::warning (this, + tr ("Sequence Manipulation Tools Plugin"),