[FFmpeg-devel] [PATCH 1/2] lavu/dict: Add new flag to allow multiple equal keys.

wm4 nfxjfg at googlemail.com
Mon Mar 28 15:56:18 CEST 2016


On Mon, 28 Mar 2016 12:05:10 +0200 (CEST)
Marton Balint <cus at passwd.hu> wrote:

> On Mon, 28 Mar 2016, Hendrik Leppkes wrote:
> 
> > People that don't speak up at all may approve silently (or at least
> > not disapprove), someone that raised issues should give an explicit
> > OK/Nevermind/whatever, agreeing with you or withdrawing his
> > complaints, there is no "silent" option there.  
> 
> This seems unreasonable, beacuse this way a silent person can block a 
> patch forever, which is not optimal.
> 
> What I usually do is after a week of silence if I feel that I addressed 
> the concerns, or there is no known way to improve the patch, I send 
> one last ping explicitly stating that I will apply the patch soon. 
> (whithin 1-2 days). And if the silence remains, I apply it.
> 

In any case I agree that my behavior wasn't ideal. I was thinking more
along the lines that I didn't really care, but someone really should
have cared. I didn't want this issue to be overlooked at least.


More information about the ffmpeg-devel mailing list