From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=66.187.233.73; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 7DF48211A2D92 for ; Mon, 11 Jun 2018 02:47:57 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id CCB9C7C6C2; Mon, 11 Jun 2018 09:47:56 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-125.rdu2.redhat.com [10.10.120.125]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5FF5E1C72D; Mon, 11 Jun 2018 09:47:56 +0000 (UTC) To: Mohammad Younas Khan Pathan Cc: edk2-devel@lists.01.org References: From: Laszlo Ersek Message-ID: <50c9dbcc-83b2-57ef-f32f-5cb0fbe166f8@redhat.com> Date: Mon, 11 Jun 2018 11:47:55 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.11.54.5 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.2]); Mon, 11 Jun 2018 09:47:56 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.2]); Mon, 11 Jun 2018 09:47:56 +0000 (UTC) for IP:'10.11.54.5' DOMAIN:'int-mx05.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'lersek@redhat.com' RCPT:'' Subject: Re: Bug 843 - Need to remove unnecessary return statements in void functions of EDKII. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 11 Jun 2018 09:47:57 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Hello Younas, On 06/11/18 09:17, Mohammad Younas Khan Pathan wrote: > Hi all, > > Please find the differences updated below, review and share your comments: (1) Please submit the patch according to the edk2 development process. https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process In particular, please don't paste a diff into whatever email client you use (e.g. gmail web interface); please use "git send-email" (or another tool that can send out pristine patches that were generated with "git format-patch"). (2) You need to add your Contributed-by and Signed-off-by lines at the end of the commit message. Please refer to "Contributions.txt" in the project root directory. In general, please write an appropriate commit message. (3) Please don't update copyright notices in source code that don't belong to you (or to your employer). For example, you update Intel copyright notices in several spots, but you don't seem to have an @intel.com email address. (4) Please split the patch to one patch per top-level directory. For example, you should have a separate patch for OvmfPkg. (5) Preferably, if the goal is to remove useless "return" statements, don't remove or add unrelated empty lines; for example in "OptionRomPkg/UndiRuntimeDxe/Init.c". It's OK to remove adjacent empty lines that had been clearly added, originally, to separate the "return" statements from the rest of the code. I suggest that we work on the OvmfPkg patch at first, to get your environment and workflow set up. Once the OvmfPkg patch looks good in all areas, you can submit the rest of the patches the same way. Thanks Laszlo