Web lists-archives.com

Re: [PATCH] coccicheck: process every source file at once




On Wed, Oct 3, 2018 at 8:52 AM Jacob Keller <jacob.keller@xxxxxxxxx> wrote:
>
> On Wed, Oct 3, 2018 at 8:05 AM SZEDER Gábor <szeder.dev@xxxxxxxxx> wrote:
> > In fact, it works finer than ever, because running 1.0.0 with this
> > patch on Travis CI notices a possible memmove() -> MOVE_ARRAY()
> > conversion:
> >
> >   https://travis-ci.org/szeder/git/jobs/436542684#L576
> >
> > Surprisingly, running 1.0.0 without this patch, or running 1.0.4
> > locally either with or without this patch doesn't notice that
> > memmove() call.  Presumably that's why Jonathan could kind-of "revert"
> > my conversion from f919ffebed (Use MOVE_ARRAY, 2018-01-22) in his
> > 6a1a79fd14 (object: move grafts to object parser, 2018-05-15) without
> > anyone noticing.
> >
>
> That seems very odd...

That looks like a bad rebase on my side as I was carrying that patch
for a while as the development of object store series took some time.

And I agree we should re-introduce the MOVE_ARRAY there.