On Tue, 11 Jun 2024 13:34:24 GMT, Sonia Zaldana Calles <szald...@openjdk.org> wrote:
>> Hi all, >> >> This PR addresses [8333685](https://bugs.openjdk.org/browse/JDK-8333685). >> >> I have added the following enhancements: >> - Removed uses of `fgrep` and `egrep` with `grep -F` and `grep -E` >> respectively. >> - Altered default behaviour to limit the processed changesets to those in >> the current branch and the current year. >> - Enabled an option to modify all changesets in the year if desired (this >> was the previous default behaviour). >> - Added named parameters and enabled `--help`. >> - Removed mercurial support. >> - Fixed a bug where copyright headers that didn't have a comma following the >> initial year of creation were not getting picked up. For example, >> [here](https://github.com/openjdk/jdk/blob/512b2b4f141f9a202984150b0427372e1a409a50/src/hotspot/cpu/zero/copy_zero.hpp#L3). >> >> - Fixed a bug where copyright headers missing the copyright symbol (c) were >> not getting picked up. Refer to the example above as well. >> >> Thanks, >> Sonia > > Sonia Zaldana Calles has updated the pull request incrementally with one > additional commit since the last revision: > > Fixing optional group to work with bsd as well Looks good to me. I haven't tried it myself though. ------------- Marked as reviewed by erikj (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/19605#pullrequestreview-2110924115