Re: [PATCH V2 0/2] MdeModulePkg PeiCore: Fix grammatical errors

["Wang, Jian J" <jian.j.wang@intel.com>]

For this patch series,

Reviewed-by: Jian J Wang <jian.j.wang@intel.com>

Regards,
Jian

> -----Original Message-----
> From: Kubacki, Michael A <michael.a.kubacki@intel.com>
> Sent: Thursday, November 28, 2019 3:58 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan <dandan.bi@intel.com>; Gao, Liming <liming.gao@intel.com>;
> Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>
> Subject: [PATCH V2 0/2] MdeModulePkg PeiCore: Fix grammatical errors
> 
> V2 Changes:
>  * "The index of the FV to search." to "Index of the FV to search"
>    in the "Instance" parameter of ProcessFvFile ()
>  * Removed "in this FV format" in the function description of
>    AddUnknownFormatFvInfo ()
>  * Explicitly stated the implied subject noun in the commit message
>    of patch 2/2
> 
> This patch series fixes grammatical errors in the PeiCore module.
> 
> Originally, this was sent as a single patch which received two
> Reviewed-by responses:
> https://edk2.groups.io/g/devel/message/50841
> 
> However, a suggestion was made to split the patch into syntactical
> and semantical changes. That suggestion is implemented in this
> series.
> 
> The Reviewed-by responses were not carried forward so the maintainers
> can evaluate this series as a whole. The previous patch has been
> dropped.
> 
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Signed-off-by: Michael Kubacki <michael.a.kubacki@intel.com>
> 
> Michael Kubacki (2):
>   MdeModulePkg PeiCore: Fix typos
>   MdeModulePkg PeiCore: Improve comment semantics
> 
>  MdeModulePkg/Core/Pei/PeiMain.inf             | 10 +--
>  MdeModulePkg/Core/Pei/FwVol/FwVol.h           | 23 +++----
>  MdeModulePkg/Core/Pei/PeiMain.h               | 49 +++++++--------
>  MdeModulePkg/Core/Pei/Dependency/Dependency.c | 12 ++--
>  MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 52 ++++++++--------
>  MdeModulePkg/Core/Pei/FwVol/FwVol.c           | 64 ++++++++++----------
>  MdeModulePkg/Core/Pei/Hob/Hob.c               |  4 +-
>  MdeModulePkg/Core/Pei/Image/Image.c           | 10 +--
>  MdeModulePkg/Core/Pei/Memory/MemoryServices.c | 18 +++---
>  MdeModulePkg/Core/Pei/PeiMain/PeiMain.c       |  2 +-
>  MdeModulePkg/Core/Pei/Ppi/Ppi.c               |  4 +-
>  MdeModulePkg/Core/Pei/Security/Security.c     | 12 ++--
>  12 files changed, 132 insertions(+), 128 deletions(-)
> 
> --
> 2.16.2.windows.1