Jon bae <jonbae77 <at> gmail.com> writes: > I hope is all right, I never did before a > git format-patch...
You used git format-patch perfectly, thank you. Your patch introduces tabs, they cannot be committed to FFmpeg. You can find tabs with "tools/patcheck", a script that is part of the source. Please do not replace av_log() with fprintf(). Please do not move opening braces, just leave them where they are now. Please keep the indentation at four spaces. Assuming that the "free" fix is unrelated to the remaining patch, please make it separate. (I don't know where FAILED() is defined and therefore cannot comment on the actual patch and if it can be simplified.) Thank you, Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel