[FFmpeg-devel] [FFmpeg-cvslog] avcodec: add XPM decoder and demuxer

Nicolas George george at nsup.org
Sun Mar 12 20:44:03 EET 2017


Le duodi 22 ventôse, an CCXXV, Paul B Mahol a écrit :
> On 3/12/17, Nicolas George <george at nsup.org> wrote:
> > Le duodi 22 ventose, an CCXXV, Paul B Mahol a ecrit :
> >> You could reply with something like: please do not push, wait for my
> >> reply.
> >
> > Yes, I COULD have done that.
> >
> > On the other hand, you SHOULD have waited to push: first replying to
> > Marton's comments, and then giving him a fair amount of time to reply.
> >
> > Pushing a simple patch when there are no comments is one thing. Pushing
> > a big patch when people have commented without replying, and even less
> > give them to follow up is not only breaking developer guidelines but
> > also basic courtesy.
> >
> >> I'm happy with code as it is.
> >
> > This is a necessary condition, but not a sufficient one.
> >
> > I will not demand you to revert, that would be silly, but please do not
> > do it again. And consider Marton's future comments (ans possibly mine)
> > exactly as binding as if the patch had not yet been applied.
> 
> OK, I'm impatiently awaiting your review.

Please acknowledge the rest of my message. Even if my own review never
arrives, the rest applies.

And if you were acting as mentor, I am sorry to say that you let a few
very dubious constructs pass.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170312/7db1e5ff/attachment.sig>


More information about the ffmpeg-devel mailing list