From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v9 2/5] ixgbe: add ops to support ethtool ops Date: Fri, 26 Jun 2015 18:34:03 -0700 Message-ID: <20150626183403.6a3dc4fb@urahara> References: <1432946276-9424-1-git-send-email-liang-min.wang@intel.com> <1435367948-20240-1-git-send-email-liang-min.wang@intel.com> <1435367948-20240-3-git-send-email-liang-min.wang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Liang-Min Larry Wang Return-path: Received: from mail-pa0-f51.google.com (mail-pa0-f51.google.com [209.85.220.51]) by dpdk.org (Postfix) with ESMTP id D0A23CA5E for ; Sat, 27 Jun 2015 03:33:57 +0200 (CEST) Received: by paceq1 with SMTP id eq1so76381615pac.3 for ; Fri, 26 Jun 2015 18:33:57 -0700 (PDT) In-Reply-To: <1435367948-20240-3-git-send-email-liang-min.wang@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, 26 Jun 2015 21:19:05 -0400 Liang-Min Larry Wang wrote: > + reg_group = reg_set[g_ind++]; > + while (reg_group) { > + count += ixgbe_regs_group_count(reg_group); > + reg_group = reg_set[g_ind++]; > + } I don't care what checkpatch says, this an example of a loop which reads better as: while ((reg_group = reg_set[g_ind++])) count += ixgbe_regs_group_count(reg_group);