[FFmpeg-devel] [RFC] replace some static with asm_visibility or so
Benoit Fouet
benoit.fouet
Wed Jan 30 09:11:22 CET 2008
Hi,
Balatoni Denes wrote:
> Hi!
>
> Tuesday 29 January 2008 19:10-kor Ivan Kalvachev ezt ?rta:
>
>> About the grammar. There is nothing worse than making your best
>> efforts to do something and then been ridiculed over the result.
>>
>
> Also well said.
>
>
this is not true...
while you're at it, why not saying that you're being ridiculed because
of code reviews ?
to me, this is the same thing: there are rules you have to follow, if
you don't, people try to correct your mistakes, it's as simple as that.
> ps: there is one thing about the high coding standards - I understand the
> need, however it would be great if in the first reply (or at least soon) it
> was made clear what is the needed minimum for svn acceptance.
could you be clearer ?
isn't the 'coding rules' section enough ?
> I mean it
> sometimes seems an endless process, when every review brings up new and new
> showstopper issues - I think it would be less frustrating to the contributor
> if all showstopper issues were made clear right from the start, it's also
> good for planning (for the contributor and everybody), to see clearly early
> in the process what needs to be fixed. just my two cents.
>
--
Ben
Purple Labs S.A.
www.purplelabs.com
More information about the ffmpeg-devel
mailing list