On Fri, Oct 30, 2015 at 01:42:30PM +0100, Paul B Mahol wrote: > On 10/30/15, Ganesh Ajjanagadde <gajja...@mit.edu> wrote: [...] > > e) I have no personal interest in asm optimizations (to address a > > suggestion on IRC). I do appreciate the hard work there, it just does > > not appeal to me for reasons that I can answer privately. I am > > interested in security aspects, algorithmic improvements, bug fixes, > > and clean up (in that order). Currently I am trying to get rid of a > > lot of the last aspect to create a foundation for myself to work on > > other things. Part of that transition is already under way - see e.g > > av_gcd patches as an algorithmic improvement. > > > > Please feel free to add things, send private email, or public > > questions so that we can come to a better understanding. > > I'm just going to tell you that you should ask before pushing something > that was not explicitly approved or even worse, was rejected.
There where alot of patches, and i mean alot of good patches but its sometimes difficult to keep track of who said what in patch review. I do think i checked and approved the log* patches that where pushed yesterday. more generally Its possible for review comments to be misunderstood, or even forgotten, both by other reviewers, the author or commiter, I do not think that this leads to any technical problem in practice as people notice very quickly when their comments have not been taken care of the way they expected What i like to ask people is to, if that happens not to assume mallice because that is neither likely nor would it be productive to assume that. If something gets ignored it is IMO basically always some misunderstanding or mistake. Now maybe there are also some simple and small technical changes that could reduce confusion. One would maybe be to add a "Approved-by:" or "Reviewed-by:" line to commit messages when the reviewer differs from the commiter and author This way one can see at a first glance who approved/reviewed each patch. I dont think this needs to be always added but maybe in cases where there is already some confusion it could maybe avoid further misuderstandings [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB DNS cache poisoning attacks, popular search engine, Google internet authority dont be evil, please
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel