About the commit subject, I think something like the following would be better: "bisect--helper: fix comparison with NULL" This way it doesn't use an uppercase for "fix", the area part is smaller, it's more specific about what the patch is doing. On Tue, May 3, 2022 at 3:15 AM Elia Pinto <gitter.spiros@xxxxxxxxx> wrote: > > Adhere to the git coding style which requires "Do not explicitly compute an > integral value with constant 0 or '\ 0', or a pointer value with constant NULL." The actual wording is: "Do not explicitly compare an integral value with constant 0 or '\0', or a pointer value with constant NULL." which is a bit different from what you wrote. (It's "compare" not "compute" and "'\0'" not "'\ 0'".) The rest looks fine.
