Re: [PATCH 2/2] docs: add headers in MyFirstObjectWalk
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Oct 30, 2021, 07:49 UTC
- Message-ID
- <CAPig+cQ2TXZtZphdkWQP9W218YUpL3mFbDo1j4Z+SjmACMfDEg@mail.gmail.com>
- In-Reply-To
- <224a256e-8700-047d-bc12-834524a3c8f3@gmail.com>
On Sat, Oct 30, 2021 at 3:33 AM Bagas Sanjaya <bagasdotme@gmail.com> wrote:
Show 6 quoted lines
> On 27/10/21 21.05, John Cai via GitGitGadget wrote: > > In several places, headers need to be included or else the code won't > > compile. Since this is the first object walk, it would be nice to > > include them in the tutorial to make it easier to follow. > > Patch title should be "#include missing headers" (dunno?)
If the series was being rerolled for some other reason, making the commit subject slightly more precise like that might make sense, but rerolling only for that reason is probably not worth the extra work for the submitter, the maintainer, and reviewers.