[Development] Qt 5.6.0 (final) packages available
Thiago Macieira
thiago.macieira at intel.com
Tue Mar 15 06:28:07 CET 2016
On terça-feira, 15 de março de 2016 00:26:06 PDT Marc Mutz wrote:
> > Not acceptable. I will only apply the patch if I understand what it does
> > and what the consequences are. I can only do that in April.
>
> I'd say it's the job of the patch *author* to describe all this in the
> commit message, not the job of the reviewer to go dig out the missing
> information by himself.
True, but as a good reviewer and especially as the maintainer, I want to
understand what the thing does. If I'm not comfortable with the change, I'm
not supposed to approve it. If I'm the maintainer, I'd be stuck with that
code.
In this specific case, it's brand, new code that I wrote recently, so I'm
supposed to know it well. And the way I designed it, deadlocks were not
supposed to happen. So I need to investigate why they're happening in the first
place.
No offence to the patch author, but I have more experience with this.
--
Thiago Macieira - thiago.macieira (AT) intel.com
Software Architect - Intel Open Source Technology Center
More information about the Development
mailing list