From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 27C7145B0C; Fri, 11 Oct 2024 05:35:39 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 120C64067A; Fri, 11 Oct 2024 05:35:39 +0200 (CEST) Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by mails.dpdk.org (Postfix) with ESMTP id A0B4840273 for ; Fri, 11 Oct 2024 05:35:37 +0200 (CEST) Received: from mail.maildlp.com (unknown [172.19.162.254]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4XPsdz50gZzfdDT; Fri, 11 Oct 2024 11:33:11 +0800 (CST) Received: from dggpeml500024.china.huawei.com (unknown [7.185.36.10]) by mail.maildlp.com (Postfix) with ESMTPS id 44ECF18010F; Fri, 11 Oct 2024 11:35:36 +0800 (CST) Received: from [10.67.121.161] (10.67.121.161) by dggpeml500024.china.huawei.com (7.185.36.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Fri, 11 Oct 2024 11:35:36 +0800 Message-ID: <5b6bc306-3e38-4252-87f5-1ba2f34a66b0@huawei.com> Date: Fri, 11 Oct 2024 11:35:35 +0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/7] app/test: introduce UT suite framework for kvargs To: Stephen Hemminger CC: , , References: <20231103095325.47843-1-fengchengwen@huawei.com> <20231103095325.47843-2-fengchengwen@huawei.com> <20241010190851.48011478@hermes.local> Content-Language: en-US From: fengchengwen In-Reply-To: <20241010190851.48011478@hermes.local> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.121.161] X-ClientProxiedBy: dggems701-chm.china.huawei.com (10.3.19.178) To dggpeml500024.china.huawei.com (7.185.36.10) X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On 2024/10/11 10:08, Stephen Hemminger wrote: > On Fri, 3 Nov 2023 09:53:19 +0000 > Chengwen Feng wrote: > >> Introduce unit test suite framework for test_kvargs.c. >> >> Signed-off-by: Chengwen Feng >> --- >> app/test/test_kvargs.c | 20 +++++++++++++------- >> 1 file changed, 13 insertions(+), 7 deletions(-) >> >> diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c >> index 7a60cac4c1..9aeb9aa0aa 100644 >> --- a/app/test/test_kvargs.c >> +++ b/app/test/test_kvargs.c >> @@ -1,5 +1,6 @@ >> /* SPDX-License-Identifier: BSD-3-Clause >> * Copyright 2014 6WIND S.A. >> + * Copyright(c) 2023 HiSilicon Limited >> */ > > The test changes look good and it is always better to have more tests. > > IANAL but starting the precedent of allowing adding copyright whenever a file > is touched just leads to lots of meaningless fluff. OK, I sent v2 which remove this line.