From owner-svn-src-head@FreeBSD.ORG Fri Jun 1 16:40:20 2012 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 3FC461065674 for ; Fri, 1 Jun 2012 16:40:20 +0000 (UTC) (envelope-from lists@eitanadler.com) Received: from mail-ob0-f182.google.com (mail-ob0-f182.google.com [209.85.214.182]) by mx1.freebsd.org (Postfix) with ESMTP id E9E0E8FC1C for ; Fri, 1 Jun 2012 16:40:19 +0000 (UTC) Received: by obcni5 with SMTP id ni5so3996610obc.13 for ; Fri, 01 Jun 2012 09:40:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=eitanadler.com; s=0xdeadbeef; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding; bh=1CyF+Xu5T4tst7j6hWiAYLUB/1KqlPyEoRf8aQ90+nQ=; b=qmtAMXAN9/hlVNEl5XeKSTqHGu5jnq7XYHW/u7/4tcSklZ9J5rDBcyE9r5ULdXP5l2 H+2KJw8YBzEjzdaD7UJxrEqgS83wSnW9dONRquCHdpWFx+Q6zejKFX4Et5zqzu+djfLD AxmzlSCj3UMXvnKVYfAIdoJgPu/ZFfwycWtIs= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding:x-gm-message-state; bh=1CyF+Xu5T4tst7j6hWiAYLUB/1KqlPyEoRf8aQ90+nQ=; b=QgF90rcGHaz4VlTZR5OBroHmWZFYhm7a7YDKKvFNHqijr5kt3O+4ukAkxmIs1xcxyG 76erpWgytGe+O+fsej7/D08P4cEEE22oU0qtSCPo6lO8bpF5Bk3++L8Iu9TyLHINXJ3G YhQYpb8ysQngU9+qlQDhpoeSVggmUfk+jEIJf9ANK/RAMhPOWLNXCMb6S0sJ9eDbCDLZ 0Z8J4XT9qPBgj7CNjjhWMADbsuVU/x1t3wcavUPIi/kSrOpZ776v8KwYBavInoA5XRE0 ToviTjUuB0e9BUiNeFrqaVmFLBU8kjPUf8WY+vkdvHD8Nrw7GS9K7r4lAGXWsMIY0POX /p0Q== Received: by 10.182.187.42 with SMTP id fp10mr3384137obc.30.1338568819232; Fri, 01 Jun 2012 09:40:19 -0700 (PDT) MIME-Version: 1.0 Sender: lists@eitanadler.com Received: by 10.182.204.69 with HTTP; Fri, 1 Jun 2012 09:39:48 -0700 (PDT) In-Reply-To: <201206011024.49122.jhb@freebsd.org> References: <201206010423.q514NKtf083043@svn.freebsd.org> <20120601150242.V1226@besplex.bde.org> <201206011024.49122.jhb@freebsd.org> From: Eitan Adler Date: Fri, 1 Jun 2012 09:39:48 -0700 X-Google-Sender-Auth: E5gQKGmPsjxQUHVQae9WAybZAQ0 Message-ID: To: John Baldwin Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Gm-Message-State: ALoCoQmGqyBxkAGZaKnsUh21A2Rrrh5ZjkCjS0LQFgg8eVBxk4vJU4N0mmQCNveiK86yyiapp/ov Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Bruce Evans Subject: Re: svn commit: r236377 - head/sys/dev/vxge/vxgehal X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 01 Jun 2012 16:40:20 -0000 On 1 June 2012 07:24, John Baldwin wrote: > This is why I personally loathe assignment side effects in boolean expres= sions > for control flow. =C2=A0I tend to write this sort of thing instead as: > > =C2=A0 =C2=A0 =C2=A0 =C2=A0channel->dtr_arr[dtr_index].dtr =3D dtrh; > =C2=A0 =C2=A0 =C2=A0 =C2=A0if (dtrh !=3D NULL) { Same here. I was told to use the assignment is style(9) by multiple people though. If it really is, I wish it would change. --=20 Eitan Adler Source & Ports committer X11, Bugbusting teams