Eric Sunshine <sunsh...@sunshineco.com> writes:
> On Tue, Sep 1, 2015 at 2:24 PM, Stefan Beller <sbel...@google.com> wrote:
>> This converts implements the helper `module_clone`. This functionality is
>
> Mentioned previously[1]: "converts implements"?
>
> [1]: http://article.gmane.org/gmane.comp.version-control.git/276966
Stefan, perhaps another round of proofreading before sending would
have helped.
I am neutral with asterisk around const myself. I think I saw this
often spelled with SP on both sides, i.e.
$ git grep ' \* const ' \*.c | wc -l
121
$ git grep ' \*const ' \*.c | wc -l
45
Majority of the latter, when viewed without pipe to wc, tells me
they are mostly borrowed code.
All the other comments from you on this patch makes sense to me.
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
