This is the mail archive of the mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 22/348] Fix -Wsahdow warnings

>>>>> "Andrey" == Andrey Smirnov <> writes:

Andrey> Initially, there were 17 patches, which, upon suggestion from Tom
Andrey> Tromey, I split so that every patch contain only changes to one
Andrey> particular function or some other small unit of the source code. I
Andrey> tend to agree with Tom that my initial decision to make only 17
Andrey> patches made it rather hard to review each, because every one of them
Andrey> contained many small but disparate changes.

I didn't really mean for you to split it down this much, but now you've
done it.  I don't want to make too much extra work for you.

Andrey> Squashing or splitting commits is not really a problem and I can do
Andrey> this, but if you want me to do so, than please point out the patches
Andrey> I should squash together.

It is hard for us to do that without seeing the whole series :)

Conversely maybe it is hard for you to know which patches are likely to
be controversial and which are obvious.

I think it is hard to discuss in the abstract.  One idea would be for
you to merge reasonably obvious patches together in a file-based way,
using your best judgment about what "reasonably obvious" means.

Or, we can just carry on.

Andrey> So given the aforementioned amount of work, can't we ignore that the
Andrey> patch count is over 9000?

Not sure what this refers to.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]