On Sat, Oct 12, 2024 at 2:07 AM Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > On Sat, Oct 12, 2024 at 12:38 AM Abhijeetsingh Meena via GitGitGadget > <gitgitgadget@xxxxxxxxx> wrote: > > + /* > > + * By default, add .git-blame-ignore-revs to the list of files > > + * containing revisions to ignore if it exists. > > + */ > > + if (access(".git-blame-ignore-revs", F_OK) == 0) { > > + string_list_append(&ignore_revs_file_list, ".git-blame-ignore-revs"); > > + } > > A couple style nits and a couple questions... One other observation: The comment above this code block doesn't say anything that isn't already stated just as clearly by the code itself. Hence, the comment adds no value, thus should be dropped.