Re: [PATCH] MdeModulePkg: Remove trailing white space

["Fu, Siyuan" <siyuan.fu@intel.com>]

Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>

> -----Original Message-----
> From: Zeng, Star
> Sent: Wednesday, September 5, 2018 10:01 AM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Bi, Dandan <dandan.bi@intel.com>;
> Gao, Liming <liming.gao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>;
> Fu, Siyuan <siyuan.fu@intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>
> Subject: [PATCH] MdeModulePkg: Remove trailing white space
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1144
> 
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Siyuan Fu <siyuan.fu@intel.com>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  MdeModulePkg/Core/Pei/Ppi/Ppi.c                    |  4 +--
>  MdeModulePkg/Include/Library/NetLib.h              |  2 +-
>  .../Library/DxeCapsuleLibFmp/DxeCapsuleLib.c       |  2 +-
>  MdeModulePkg/Library/DxeNetLib/DxeNetLib.c         |  4 +--
>  MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c        |  2 +-
>  MdeModulePkg/Universal/Network/Ip4Dxe/Ip4If.c      | 14 +++++-----
>  MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Impl.c    | 30 +++++++++++------
> -----
>  7 files changed, 29 insertions(+), 29 deletions(-)
> 
> diff --git a/MdeModulePkg/Core/Pei/Ppi/Ppi.c
> b/MdeModulePkg/Core/Pei/Ppi/Ppi.c
> index d8ba2dd42b59..6f03858b8a94 100644
> --- a/MdeModulePkg/Core/Pei/Ppi/Ppi.c
> +++ b/MdeModulePkg/Core/Pei/Ppi/Ppi.c
> @@ -143,7 +143,7 @@ ConvertPointerInRanges (
> 
>    Migrate Single PPI Pointer from the temporary memory to PEI installed
> memory.
> 
> -  @param SecCoreData     Points to a data structure containing SEC to PEI
> handoff data, such as the size
> +  @param SecCoreData     Points to a data structure containing SEC to PEI
> handoff data, such as the size
>                           and location of temporary RAM, the stack
> location and the BFV location.
>    @param PrivateData     Pointer to PeiCore's private data structure.
>    @param PpiPointer      Pointer to Ppi
> @@ -179,7 +179,7 @@ ConvertSinglePpiPointer (
> 
>    Migrate PPI Pointers from the temporary memory to PEI installed memory.
> 
> -  @param SecCoreData     Points to a data structure containing SEC to PEI
> handoff data, such as the size
> +  @param SecCoreData     Points to a data structure containing SEC to PEI
> handoff data, such as the size
>                           and location of temporary RAM, the stack
> location and the BFV location.
>    @param PrivateData     Pointer to PeiCore's private data structure.
> 
> diff --git a/MdeModulePkg/Include/Library/NetLib.h
> b/MdeModulePkg/Include/Library/NetLib.h
> index b7ef99c7b568..09779739213e 100644
> --- a/MdeModulePkg/Include/Library/NetLib.h
> +++ b/MdeModulePkg/Include/Library/NetLib.h
> @@ -424,7 +424,7 @@ NetGetIpClass (
>    except when the originator is one of the endpoints of a point-to-point
> link with a 31-bit
>    mask (RFC3021), or a 32bit NetMask (all 0xFF) is used for special
> network environment (e.g.
>    PPP link).
> -
> +
>    @param[in]  Ip                    The IP to check against.
>    @param[in]  NetMask               The mask of the IP.
> 
> diff --git a/MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c
> b/MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c
> index 91c6849a4658..20fd29a3a87f 100644
> --- a/MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c
> +++ b/MdeModulePkg/Library/DxeCapsuleLibFmp/DxeCapsuleLib.c
> @@ -765,7 +765,7 @@ GetFmpHandleBufferByType (
>              MatchedHandleBuffer[MatchedNumberOfHandles] =
> HandleBuffer[Index];
>            }
>            if (MatchedResetRequiredBuffer != NULL) {
> -            MatchedResetRequiredBuffer[MatchedNumberOfHandles] =
> (((TempFmpImageInfo->AttributesSupported &
> +            MatchedResetRequiredBuffer[MatchedNumberOfHandles] =
> (((TempFmpImageInfo->AttributesSupported &
> 
> IMAGE_ATTRIBUTE_RESET_REQUIRED) != 0) &&
> 
> ((TempFmpImageInfo->AttributesSetting &
> 
> IMAGE_ATTRIBUTE_RESET_REQUIRED) != 0));
> diff --git a/MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
> b/MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
> index 63f4724062e9..9e3ea2490928 100644
> --- a/MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
> +++ b/MdeModulePkg/Library/DxeNetLib/DxeNetLib.c
> @@ -656,7 +656,7 @@ NetGetIpClass (
>    except when the originator is one of the endpoints of a point-to-point
> link with a 31-bit
>    mask (RFC3021), or a 32bit NetMask (all 0xFF) is used for special
> network environment (e.g.
>    PPP link).
> -
> +
>    @param[in]  Ip                    The IP to check against.
>    @param[in]  NetMask               The mask of the IP.
> 
> @@ -671,7 +671,7 @@ NetIp4IsUnicast (
>    )
>  {
>    INTN   MaskLength;
> -
> +
>    ASSERT (NetMask != 0);
> 
>    if (Ip == 0 || IP4_IS_LOCAL_BROADCAST (Ip)) {
> diff --git a/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
> b/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
> index 219059760e97..129e90ca11b4 100644
> --- a/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
> +++ b/MdeModulePkg/Universal/EsrtFmpDxe/EsrtFmp.c
> @@ -296,7 +296,7 @@ CreateFmpBasedEsrt (
> 
>    Table->FwResourceCount    = 0;
>    Table->FwResourceCountMax = GROWTH_STEP;
> -  Table->FwResourceVersion  =
> EFI_SYSTEM_RESOURCE_TABLE_FIRMWARE_RESOURCE_VERSION;
> +  Table->FwResourceVersion  =
> EFI_SYSTEM_RESOURCE_TABLE_FIRMWARE_RESOURCE_VERSION;
> 
>    for (Index = 0; Index < NoProtocols; Index++) {
>      Fmp = (EFI_FIRMWARE_MANAGEMENT_PROTOCOL *) Buffer[Index];
> diff --git a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4If.c
> b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4If.c
> index 499a486a8b5d..4e3988132d6b 100644
> --- a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4If.c
> +++ b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4If.c
> @@ -796,14 +796,14 @@ Ip4FreeInterface (
>  }
> 
>  /**
> -  This function tries to send all the queued frames in ArpQue to the
> default gateway if
> +  This function tries to send all the queued frames in ArpQue to the
> default gateway if
>    the ARP resolve for direct destination address is failed when using /32
> subnet mask.
> 
>    @param[in]   ArpQue           The ARP queue of a failed request.
> -
> +
>    @retval EFI_SUCCESS           All the queued frames have been send to
> the default route.
>    @retval Others                Failed to send the queued frames.
> -
> +
>  **/
>  EFI_STATUS
>  Ip4SendFrameToDefaultRoute (
> @@ -817,7 +817,7 @@ Ip4SendFrameToDefaultRoute (
>    IP4_ADDR                  Gateway;
>    EFI_STATUS                Status;
>    IP4_ROUTE_ENTRY           *DefaultRoute;
> -
> +
>    //
>    // ARP resolve failed when using /32 subnet mask.
>    //
> @@ -850,7 +850,7 @@ Ip4SendFrameToDefaultRoute (
>      Gateway = DefaultRoute->NextHop;
>      if (ArpQue->Ip == Gateway) {
>        //
> -      // ARP resolve for the default route is failed, return error to
> caller.
> +      // ARP resolve for the default route is failed, return error to
> caller.
>        //
>        Status= EFI_NO_MAPPING;
>        goto ON_ERROR;
> @@ -865,7 +865,7 @@ Ip4SendFrameToDefaultRoute (
>    }
> 
>    return EFI_SUCCESS;
> -
> +
>  ON_ERROR:
>    if (RtCacheEntry != NULL) {
>      Ip4FreeRouteCacheEntry (RtCacheEntry);
> @@ -905,7 +905,7 @@ Ip4OnArpResolvedDpc (
>    RemoveEntryList (&ArpQue->Link);
> 
>    //
> -  // ARP resolve failed for some reason.
> +  // ARP resolve failed for some reason.
>    //
>    if (NET_MAC_EQUAL (&ArpQue->Mac, &mZeroMacAddress, ArpQue->Interface-
> >HwaddrLen)) {
>      if (ArpQue->Interface->SubnetMask != IP4_ALLONE_ADDRESS) {
> diff --git a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Impl.c
> b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Impl.c
> index 7c27db6753d0..87aadf13e470 100644
> --- a/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Impl.c
> +++ b/MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Impl.c
> @@ -682,7 +682,7 @@ Ip4ConfigProtocol (
>      //
>      if (IpSb->State == IP4_SERVICE_UNSTARTED) {
>        //
> -      // Trigger the EFI_IP4_CONFIG2_PROTOCOL to retrieve the
> +      // Trigger the EFI_IP4_CONFIG2_PROTOCOL to retrieve the
>        // default IPv4 address if it is not available yet.
>        //
>        Policy = IpSb->Ip4Config2Instance.Policy;
> @@ -1660,7 +1660,7 @@ EfiIp4Transmit (
>      }
> 
>      RawHdrLen = (UINT8) (RawHdrLen << 2);
> -
> +
>      CopyMem (&Head, FirstFragment, IP4_MIN_HEADLEN);
> 
>      Ip4NtohHead (&Head);
> @@ -2250,8 +2250,8 @@ Ip4SentPacketTicking (
>  }
> 
>  /**
> -  This heart beat timer of IP4 service instance times out all of its IP4
> children's
> -  received-but-not-delivered and transmitted-but-not-recycle packets, and
> provides
> +  This heart beat timer of IP4 service instance times out all of its IP4
> children's
> +  received-but-not-delivered and transmitted-but-not-recycle packets, and
> provides
>    time input for its IGMP protocol.
> 
>    @param[in]  Event                  The IP4 service instance's heart
> beat timer.
> @@ -2269,17 +2269,17 @@ Ip4TimerTicking (
> 
>    IpSb = (IP4_SERVICE *) Context;
>    NET_CHECK_SIGNATURE (IpSb, IP4_SERVICE_SIGNATURE);
> -
> +
>    Ip4PacketTimerTicking (IpSb);
>    Ip4IgmpTicking (IpSb);
>  }
> 
>  /**
> -  This dedicated timer is used to poll underlying network media status.
> In case
> -  of cable swap or wireless network switch, a new round auto
> configuration will
> -  be initiated. The timer will signal the IP4 to run DHCP configuration
> again.
> -  IP4 driver will free old IP address related resource, such as route
> table and
> -  Interface, then initiate a DHCP process to acquire new IP, eventually
> create
> +  This dedicated timer is used to poll underlying network media status.
> In case
> +  of cable swap or wireless network switch, a new round auto
> configuration will
> +  be initiated. The timer will signal the IP4 to run DHCP configuration
> again.
> +  IP4 driver will free old IP address related resource, such as route
> table and
> +  Interface, then initiate a DHCP process to acquire new IP, eventually
> create
>    route table for new IP address.
> 
>    @param[in]  Event                  The IP4 service instance's heart
> beat timer.
> @@ -2300,12 +2300,12 @@ Ip4TimerReconfigChecking (
> 
>    IpSb = (IP4_SERVICE *) Context;
>    NET_CHECK_SIGNATURE (IpSb, IP4_SERVICE_SIGNATURE);
> -
> +
>    OldMediaPresent = IpSb->MediaPresent;
> 
>    //
> -  // Get fresh mode data from MNP, since underlying media status may
> change.
> -  // Here, it needs to mention that the MediaPresent can also be checked
> even if
> +  // Get fresh mode data from MNP, since underlying media status may
> change.
> +  // Here, it needs to mention that the MediaPresent can also be checked
> even if
>    // EFI_NOT_STARTED returned while this MNP child driver instance isn't
> configured.
>    //
>    Status = IpSb->Mnp->GetModeData (IpSb->Mnp, NULL, &SnpModeData);
> @@ -2320,8 +2320,8 @@ Ip4TimerReconfigChecking (
>    if (!OldMediaPresent && IpSb->MediaPresent && (IpSb-
> >Ip4Config2Instance.Policy == Ip4Config2PolicyDhcp)) {
>      //
>      // Signal the IP4 to run the dhcp configuration again. IP4 driver
> will free
> -    // old IP address related resource, such as route table and Interface,
> then
> -    // initiate a DHCP round to acquire new IP, eventually
> +    // old IP address related resource, such as route table and Interface,
> then
> +    // initiate a DHCP round to acquire new IP, eventually
>      // create route table for new IP address.
>      //
>      if (IpSb->ReconfigEvent != NULL) {
> --
> 2.7.0.windows.1