[FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

Nicolas George george at nsup.org
Wed Jul 30 23:48:47 CEST 2014


Le duodi 12 thermidor, an CCXXII, Derek Buitenhuis a écrit :
> Yeah, why should you have to explain the reason for a change in
> a review of that change? How silly. I should just divine the reason
> why. I guess we don't count as "everyone".

To review a change, the minimum is to read it. The answers to the first
questions were in the proposed commit messages. I hope you'll excuse me if I
take offence when someone nags me with questions while they obviously did
not care enough to read them.

If you have read them and would like details or find something unclear,
OTOH, I would gladly explain what needs explaining.


New patch series sent, with the constructive comments hopefully taken into
account.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140730/83f179dc/attachment.asc>


More information about the ffmpeg-devel mailing list