From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id D5A69A00BE for ; Wed, 29 Apr 2020 01:16:35 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id CAA451D63F; Wed, 29 Apr 2020 01:16:35 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 201C31D63E; Wed, 29 Apr 2020 01:16:34 +0200 (CEST) In-Reply-To: <20200428231415.17985-4-stephen@networkplumber.org> References: <20200428231415.17985-4-stephen@networkplumber.org> To: test-report@dpdk.org Cc: Stephen Hemminger Message-Id: <20200428231634.201C31D63E@dpdk.org> Date: Wed, 29 Apr 2020 01:16:34 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw69472 [PATCH v2 3/8] eal: alarm: close file on cleanup X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/69472 _coding style issues_ WARNING:LEADING_SPACE: please, no spaces at the start of a line #135: FILE: lib/librte_eal/freebsd/eal_alarm.c:67: + if (intr_handle.fd == -1)$ WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15) #135: FILE: lib/librte_eal/freebsd/eal_alarm.c:67: + if (intr_handle.fd == -1) + return; ERROR:CODE_INDENT: code indent should use tabs where possible #136: FILE: lib/librte_eal/freebsd/eal_alarm.c:68: + return;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #136: FILE: lib/librte_eal/freebsd/eal_alarm.c:68: + return;$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #138: FILE: lib/librte_eal/freebsd/eal_alarm.c:70: + close(intr_handle.fd);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #139: FILE: lib/librte_eal/freebsd/eal_alarm.c:71: + intr_handle.fd = -1;$ total: 1 errors, 5 warnings, 66 lines checked