From owner-freebsd-current@FreeBSD.ORG Sun Jun 30 14:14:35 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 3A393ED9 for ; Sun, 30 Jun 2013 14:14:35 +0000 (UTC) (envelope-from dt71@gmx.com) Received: from mout.gmx.net (mout.gmx.net [212.227.15.18]) by mx1.freebsd.org (Postfix) with ESMTP id B73D81F98 for ; Sun, 30 Jun 2013 14:14:34 +0000 (UTC) Received: from [192.168.1.80] ([84.2.254.103]) by mail.gmx.com (mrgmx103) with ESMTPSA (Nemesis) id 0M8MyE-1TyqCJ2a1K-00vzUq for ; Sun, 30 Jun 2013 16:14:34 +0200 Message-ID: <51D03D27.3020100@gmx.com> Date: Sun, 30 Jun 2013 16:13:59 +0200 From: dt71@gmx.com User-Agent: Mozilla/5.0 (X11; FreeBSD i386; rv:20.0) Gecko/20100101 Firefox/20.0 SeaMonkey/2.17.1 MIME-Version: 1.0 To: freebsd-current@freebsd.org Subject: Re: another -Wunsequenced topic References: <51CEEC34.2010308@gmx.com> In-Reply-To: <51CEEC34.2010308@gmx.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:AFFIDXGf9XRUaJW8us+D9WpDNJYXbgAuK6tRhMqRwjLc6GVUlm8 8OGAyJJSsFkCrdUvGb4XdOtY9ukVRvbu8IWQbE4W1FJCloS4/J19gqXMPia8qIbuhNNMq7n A/N9ZJykeU/RFOqvBcbsUSc9Hep/yEjAj+CWcitJHwiG/9GTgTskQRn5zTQKnp8Usa/jlw9 p3wpurpSE3LrzV/3ayy/Q== X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 30 Jun 2013 14:14:35 -0000 There are more. Take the first hunk with caution. Index: contrib/gdb/gdb/dwarf2-frame.c =================================================================== --- contrib/gdb/gdb/dwarf2-frame.c (revision 252384) +++ contrib/gdb/gdb/dwarf2-frame.c (working copy) @@ -1361,7 +1361,7 @@ else if (*augmentation == 'P') { /* Skip. */ - buf += size_of_encoded_value (*buf++); + buf += size_of_encoded_value (*buf) + 1; augmentation++; } Index: usr.sbin/moused/moused.c =================================================================== --- usr.sbin/moused/moused.c (revision 252384) +++ usr.sbin/moused/moused.c (working copy) @@ -2455,7 +2455,7 @@ return (FALSE); lbutton = atoi(s); - arg = skipspace(++arg); + arg = skipspace(arg + 1); s = arg; while (isdigit(*arg)) ++arg; Index: lib/libstand/nfs.c =================================================================== --- lib/libstand/nfs.c (revision 252384) +++ lib/libstand/nfs.c (working copy) @@ -1465,8 +1465,9 @@ d->d_name[d->d_namlen] = '\0'; pos = roundup(d->d_namlen, sizeof(uint32_t)) / sizeof(uint32_t); - fp->off = cookie = ((uint64_t)ntohl(rent->nameplus[pos++]) << 32) | - ntohl(rent->nameplus[pos++]); + fp->off = cookie = ((uint64_t)ntohl(rent->nameplus[pos]) << 32) | + ntohl(rent->nameplus[pos + 1]); + pos += 2; buf = (u_char *)&rent->nameplus[pos]; return (0); } Index: contrib/sendmail/src/recipient.c =================================================================== --- contrib/sendmail/src/recipient.c (revision 252384) +++ contrib/sendmail/src/recipient.c (working copy) @@ -1834,7 +1834,7 @@ /* #@# introduces a comment anywhere */ /* for Japanese character sets */ - for (p = buf; (p = strchr(++p, '#')) != NULL; ) + for (p = buf; (p = strchr(p + 1, '#')) != NULL; ) { if (p[1] == '@' && p[2] == '#' && isascii(p[-1]) && isspace(p[-1]) && Index: usr.sbin/pkg_install/create/perform.c =================================================================== --- usr.sbin/pkg_install/create/perform.c (revision 252384) +++ usr.sbin/pkg_install/create/perform.c (working copy) @@ -149,7 +149,7 @@ /* Count number of dependencies */ for (cp = Pkgdeps; cp != NULL && *cp != '\0'; - cp = strpbrk(++cp, " \t\n")) { + cp = strpbrk(cp + 1, " \t\n")) { ndeps++; }