Hi Ray, On 2/23/24 03:35, Ni, Ray wrote: >> + @param[in] MpHandOff Previous HOB body. Pass NULL to get the >> first HOB. > > Can you replace "...HOB Body. Pass NULL..." with "...HOB Body. Pass NULL..."? > Your current comments contain two spaces. > > With that, Reviewed-by: Ray Ni <ray...@intel.com>
The two spaces after the dot are valid English -- it's called "English spacing after a full stop": https://en.wikipedia.org/wiki/Full_stop#Spacing_after_a_full_stop Some projects use English spacing in comments and documentation, so a developer working on multiple projects may not find it easy to switch between French and English spacing. (If you check Gerd's commit messages in this series, those also use English spacing.) I could edit out the second spaces before merging this series, but I think that wouldn't be right; again, it's valid English. (E.g., we also don't try to enforce American vs. British English spelling in comments, even though the American one is much more common in the codebase.) Reviewed-by: Laszlo Ersek <ler...@redhat.com> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115957): https://edk2.groups.io/g/devel/message/115957 Mute This Topic: https://groups.io/mt/104510907/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
