From owner-svn-ports-head@freebsd.org Sun Jan 29 21:39:31 2017 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 831A5CC7649; Sun, 29 Jan 2017 21:39:31 +0000 (UTC) (envelope-from grembo@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::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 5DCEF1FF0; Sun, 29 Jan 2017 21:39:31 +0000 (UTC) (envelope-from grembo@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v0TLdUoK092095; Sun, 29 Jan 2017 21:39:30 GMT (envelope-from grembo@FreeBSD.org) Received: (from grembo@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v0TLdUOj092094; Sun, 29 Jan 2017 21:39:30 GMT (envelope-from grembo@FreeBSD.org) Message-Id: <201701292139.v0TLdUOj092094@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: grembo set sender to grembo@FreeBSD.org using -f From: Michael Gmelin Date: Sun, 29 Jan 2017 21:39:30 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r432801 - head/devel/ice/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.23 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, 29 Jan 2017 21:39:31 -0000 Author: grembo Date: Sun Jan 29 21:39:30 2017 New Revision: 432801 URL: https://svnweb.freebsd.org/changeset/ports/432801 Log: Allow building devel/py-ice with clang 4.0, based on bug report Tested that this still builds fine using previous versions of clang, didn't bump revision, as there is no functional change and it's a build time error. PR: 216356 Reported by: jbeich Added: head/devel/ice/files/patch-python-modules-IcePy-Operation.cpp (contents, props changed) Added: head/devel/ice/files/patch-python-modules-IcePy-Operation.cpp ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/devel/ice/files/patch-python-modules-IcePy-Operation.cpp Sun Jan 29 21:39:30 2017 (r432801) @@ -0,0 +1,186 @@ +--- python/modules/IcePy/Operation.cpp.orig 2017-01-29 22:23:51.186130998 +0100 ++++ python/modules/IcePy/Operation.cpp 2017-01-29 22:24:55.929130998 +0100 +@@ -1682,11 +1682,11 @@ + } + else + { +- if(PyTuple_SET_ITEM(results.get(), info->pos, Unset) < 0) ++ if(PyTuple_SetItem(results.get(), info->pos, Unset) < 0) + { + return 0; + } +- Py_INCREF(Unset); // PyTuple_SET_ITEM steals a reference. ++ Py_INCREF(Unset); // PyTuple_SetItem steals a reference. + } + } + +@@ -2546,7 +2546,7 @@ + throwPythonException(); + } + +- if(PyTuple_SET_ITEM(result.get(), 0, ok ? incTrue() : incFalse()) < 0) ++ if(PyTuple_SetItem(result.get(), 0, ok ? incTrue() : incFalse()) < 0) + { + throwPythonException(); + } +@@ -2586,11 +2586,11 @@ + } + #endif + +- if(PyTuple_SET_ITEM(result.get(), 1, op.get()) < 0) ++ if(PyTuple_SetItem(result.get(), 1, op.get()) < 0) + { + throwPythonException(); + } +- op.release(); // PyTuple_SET_ITEM steals a reference. ++ op.release(); // PyTuple_SetItem steals a reference. + + return result.release(); + } +@@ -2827,7 +2827,7 @@ + return 0; + } + +- if(PyTuple_SET_ITEM(args.get(), 0, ok ? incTrue() : incFalse()) < 0) ++ if(PyTuple_SetItem(args.get(), 0, ok ? incTrue() : incFalse()) < 0) + { + return 0; + } +@@ -2867,11 +2867,11 @@ + memcpy(buf, results.first, sz); + #endif + +- if(PyTuple_SET_ITEM(args.get(), 1, op.get()) < 0) ++ if(PyTuple_SetItem(args.get(), 1, op.get()) < 0) + { + return 0; + } +- op.release(); // PyTuple_SET_ITEM steals a reference. ++ op.release(); // PyTuple_SetItem steals a reference. + + return args.release(); + } +@@ -2910,7 +2910,7 @@ + return; + } + +- if(PyTuple_SET_ITEM(args.get(), 0, ok ? incTrue() : incFalse()) < 0) ++ if(PyTuple_SetItem(args.get(), 0, ok ? incTrue() : incFalse()) < 0) + { + assert(PyErr_Occurred()); + PyErr_Print(); +@@ -2958,13 +2958,13 @@ + memcpy(buf, results.first, sz); + #endif + +- if(PyTuple_SET_ITEM(args.get(), 1, op.get()) < 0) ++ if(PyTuple_SetItem(args.get(), 1, op.get()) < 0) + { + assert(PyErr_Occurred()); + PyErr_Print(); + return; + } +- op.release(); // PyTuple_SET_ITEM steals a reference. ++ op.release(); // PyTuple_SetItem steals a reference. + + PyObjectHandle tmp = PyObject_Call(_response, args.get(), 0); + if(PyErr_Occurred()) +@@ -3126,7 +3126,7 @@ + return; + } + +- if(PyTuple_SET_ITEM(args.get(), 0, ok ? incTrue() : incFalse()) < 0) ++ if(PyTuple_SetItem(args.get(), 0, ok ? incTrue() : incFalse()) < 0) + { + assert(PyErr_Occurred()); + PyErr_Print(); +@@ -3174,13 +3174,13 @@ + memcpy(buf, results.first, sz); + #endif + +- if(PyTuple_SET_ITEM(args.get(), 1, op.get()) < 0) ++ if(PyTuple_SetItem(args.get(), 1, op.get()) < 0) + { + assert(PyErr_Occurred()); + PyErr_Print(); + return; + } +- op.release(); // PyTuple_SET_ITEM steals a reference. ++ op.release(); // PyTuple_SetItem steals a reference. + + const string methodName = "ice_response"; + if(!PyObject_HasAttrString(_callback, STRCAST(methodName.c_str()))) +@@ -3302,11 +3302,11 @@ + } + else + { +- if(PyTuple_SET_ITEM(args.get(), info->pos + offset, Unset) < 0) ++ if(PyTuple_SetItem(args.get(), info->pos + offset, Unset) < 0) + { + throwPythonException(); + } +- Py_INCREF(Unset); // PyTuple_SET_ITEM steals a reference. ++ Py_INCREF(Unset); // PyTuple_SetItem steals a reference. + } + } + +@@ -3329,11 +3329,11 @@ + // Create an object to represent Ice::Current. We need to append this to the argument tuple. + // + PyObjectHandle curr = createCurrent(current); +- if(PyTuple_SET_ITEM(args.get(), PyTuple_GET_SIZE(args.get()) - 1, curr.get()) < 0) ++ if(PyTuple_SetItem(args.get(), PyTuple_GET_SIZE(args.get()) - 1, curr.get()) < 0) + { + throwPythonException(); + } +- curr.release(); // PyTuple_SET_ITEM steals a reference. ++ curr.release(); // PyTuple_SetItem steals a reference. + + if(_op->amd) + { +@@ -3347,7 +3347,7 @@ + } + obj->upcall = new UpcallPtr(this); + obj->encoding = current.encoding; +- if(PyTuple_SET_ITEM(args.get(), 0, (PyObject*)obj) < 0) // PyTuple_SET_ITEM steals a reference. ++ if(PyTuple_SetItem(args.get(), 0, (PyObject*)obj) < 0) // PyTuple_SetItem steals a reference. + { + Py_DECREF(obj); + throwPythonException(); +@@ -3698,23 +3698,23 @@ + } + #endif + +- if(PyTuple_SET_ITEM(args.get(), start, ip.get()) < 0) ++ if(PyTuple_SetItem(args.get(), start, ip.get()) < 0) + { + throwPythonException(); + } + ++start; +- ip.release(); // PyTuple_SET_ITEM steals a reference. ++ ip.release(); // PyTuple_SetItem steals a reference. + + // + // Create an object to represent Ice::Current. We need to append + // this to the argument tuple. + // + PyObjectHandle curr = createCurrent(current); +- if(PyTuple_SET_ITEM(args.get(), start, curr.get()) < 0) ++ if(PyTuple_SetItem(args.get(), start, curr.get()) < 0) + { + throwPythonException(); + } +- curr.release(); // PyTuple_SET_ITEM steals a reference. ++ curr.release(); // PyTuple_SetItem steals a reference. + + string dispatchName = "ice_invoke"; + if(_amd) +@@ -3730,7 +3730,7 @@ + } + obj->upcall = new UpcallPtr(this); + obj->encoding = current.encoding; +- if(PyTuple_SET_ITEM(args.get(), 0, (PyObject*)obj) < 0) // PyTuple_SET_ITEM steals a reference. ++ if(PyTuple_SetItem(args.get(), 0, (PyObject*)obj) < 0) // PyTuple_SetItem steals a reference. + { + Py_DECREF(obj); + throwPythonException();