From: "Maciej Rabeda" <maciej.rabeda@linux.intel.com>
To: Wenyi Xie <xiewenyi2@huawei.com>,
devel@edk2.groups.io, jiaxin.wu@intel.com, siyuan.fu@intel.com
Cc: songdongkuang@huawei.com
Subject: Re: [PATCH EDK2 v1 1/1] NetworkPkg/DnsDxe: cosmetic fixups
Date: Tue, 5 Jan 2021 15:16:05 +0100 [thread overview]
Message-ID: <98e26a3d-6d64-0f7b-f2ac-ea75ffee9cde@linux.intel.com> (raw)
In-Reply-To: <1607065503-29304-2-git-send-email-xiewenyi2@huawei.com>
Thanks for the patch!
Reviewed-by: Maciej Rabeda <maciej.rabeda@linux.intel.com>
On 04-Dec-20 08:05, Wenyi Xie wrote:
> add whitespace according to coding style.
>
> Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Cc: Siyuan Fu <siyuan.fu@intel.com>
> Signed-off-by: Wenyi Xie <xiewenyi2@huawei.com>
> ---
> NetworkPkg/DnsDxe/DnsImpl.h | 2 +-
> NetworkPkg/DnsDxe/DnsDhcp.c | 4 ++--
> NetworkPkg/DnsDxe/DnsDriver.c | 4 ++--
> NetworkPkg/DnsDxe/DnsImpl.c | 8 ++++----
> NetworkPkg/DnsDxe/DnsProtocol.c | 4 ++--
> 5 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/NetworkPkg/DnsDxe/DnsImpl.h b/NetworkPkg/DnsDxe/DnsImpl.h
> index 10bc399e9ed5..affbb4be8ded 100644
> --- a/NetworkPkg/DnsDxe/DnsImpl.h
> +++ b/NetworkPkg/DnsDxe/DnsImpl.h
> @@ -125,7 +125,7 @@ typedef struct {
> EFI_DNS6_COMPLETION_TOKEN *Token;
> } DNS6_TOKEN_ENTRY;
>
> -union _DNS_FLAGS{
> +union _DNS_FLAGS {
> struct {
> UINT16 RCode:4;
> UINT16 Zero:3;
> diff --git a/NetworkPkg/DnsDxe/DnsDhcp.c b/NetworkPkg/DnsDxe/DnsDhcp.c
> index 9ea9d7c8f674..8f8b2aab1e32 100644
> --- a/NetworkPkg/DnsDxe/DnsDhcp.c
> +++ b/NetworkPkg/DnsDxe/DnsDhcp.c
> @@ -131,7 +131,7 @@ ParseDhcp4Ack (
> return EFI_OUT_OF_RESOURCES;
> }
>
> - for(Count=0; Count < ServerCount; Count++){
> + for (Count=0; Count < ServerCount; Count++) {
> CopyMem (ServerList + Count, &OptionList[Index]->Data[4 * Count], sizeof (EFI_IPv4_ADDRESS));
> }
>
> @@ -225,7 +225,7 @@ ParseDhcp6Ack (
> return EFI_OUT_OF_RESOURCES;
> }
>
> - for(Count=0; Count < ServerCount; Count++){
> + for (Count=0; Count < ServerCount; Count++) {
> CopyMem (ServerList + Count, &OptionList[Index]->Data[16 * Count], sizeof (EFI_IPv6_ADDRESS));
> }
>
> diff --git a/NetworkPkg/DnsDxe/DnsDriver.c b/NetworkPkg/DnsDxe/DnsDriver.c
> index f099da8a5a95..4db0c6bdd1eb 100644
> --- a/NetworkPkg/DnsDxe/DnsDriver.c
> +++ b/NetworkPkg/DnsDxe/DnsDriver.c
> @@ -197,11 +197,11 @@ DnsDestroyService (
> {
> UdpIoFreeIo (DnsSb->ConnectUdp);
>
> - if (DnsSb->TimerToGetMap != NULL){
> + if (DnsSb->TimerToGetMap != NULL) {
> gBS->CloseEvent (DnsSb->TimerToGetMap);
> }
>
> - if (DnsSb->Timer != NULL){
> + if (DnsSb->Timer != NULL) {
> gBS->CloseEvent (DnsSb->Timer);
> }
>
> diff --git a/NetworkPkg/DnsDxe/DnsImpl.c b/NetworkPkg/DnsDxe/DnsImpl.c
> index 25628a5704f0..2edcb280ac4d 100644
> --- a/NetworkPkg/DnsDxe/DnsImpl.c
> +++ b/NetworkPkg/DnsDxe/DnsImpl.c
> @@ -799,7 +799,7 @@ UpdateDns4Cache (
> Item->DnsCache.Timeout = DnsCacheEntry.Timeout;
>
> return EFI_SUCCESS;
> - }else {
> + } else {
> return EFI_ACCESS_DENIED;
> }
> }
> @@ -897,7 +897,7 @@ UpdateDns6Cache (
> Item->DnsCache.Timeout = DnsCacheEntry.Timeout;
>
> return EFI_SUCCESS;
> - }else {
> + } else {
> return EFI_ACCESS_DENIED;
> }
> }
> @@ -1080,7 +1080,7 @@ IsValidDnsResponse (
> NET_LIST_FOR_EACH (Entry, &TokensMap->Used) {
> *Item = NET_LIST_USER_STRUCT (Entry, NET_MAP_ITEM, Link);
> Packet = (NET_BUF *) ((*Item)->Value);
> - if (Packet == NULL){
> + if (Packet == NULL) {
>
> continue;
> } else {
> @@ -2127,7 +2127,7 @@ DnsOnTimerRetransmit (
> }
> }
> }
> - }else {
> + } else {
> //
> // Iterate through all the children of the DNS service instance. Time
> // out the packet. If maximum retries reached, clean the Token up.
> diff --git a/NetworkPkg/DnsDxe/DnsProtocol.c b/NetworkPkg/DnsDxe/DnsProtocol.c
> index 4acc5fca46cf..a0bb58ef1db3 100644
> --- a/NetworkPkg/DnsDxe/DnsProtocol.c
> +++ b/NetworkPkg/DnsDxe/DnsProtocol.c
> @@ -217,7 +217,7 @@ Dns4Configure (
> Dns4InstanceCancelToken(Instance, NULL);
> }
>
> - if (Instance->UdpIo != NULL){
> + if (Instance->UdpIo != NULL) {
> UdpIoCleanIo (Instance->UdpIo);
> }
>
> @@ -1061,7 +1061,7 @@ Dns6Configure (
> Dns6InstanceCancelToken(Instance, NULL);
> }
>
> - if (Instance->UdpIo != NULL){
> + if (Instance->UdpIo != NULL) {
> UdpIoCleanIo (Instance->UdpIo);
> }
>
next prev parent reply other threads:[~2021-01-05 14:16 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-04 7:05 [PATCH EDK2 v1 0/1] NetworkPkg/DnsDxe: cosmetic fixups wenyi,xie
2020-12-04 7:05 ` [PATCH EDK2 v1 1/1] " wenyi,xie
2021-01-05 14:16 ` Maciej Rabeda [this message]
[not found] ` <16575BAFF098A13A.29174@groups.io>
2021-01-05 14:44 ` [edk2-devel] " Maciej Rabeda
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=98e26a3d-6d64-0f7b-f2ac-ea75ffee9cde@linux.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox