[Ffmpeg-devel] [RFC] ffmpeg-windows mailinglist?
Michael Niedermayer
michaelni
Wed Aug 2 10:20:53 CEST 2006
Hi
On Wed, Aug 02, 2006 at 12:28:13AM -0500, Chris Forsythe wrote:
> Rich Felker wrote:
>
> >On Tue, Aug 01, 2006 at 09:09:21PM -0400, Augie Fackler wrote:
> >
> >
> >>Usually the comments boil down to "this is ugly" or something similar
> >>with no feedback on what could make the patch "pretty" enough to be
> >>accepted. As an example, taken from <http://svn.cod3r.com/perian/
> >>trunk/ffmpeg-svn-mactel.patch>:
> >>- ".balign 16 \n\t"
> >>+ BALIGN_16
> >>and
> >>+#if defined(__APPLE__)
> >>+# define BALIGN_8 ".align 3 \n\t"
> >>+# define BALIGN_16 ".align 4 \n\t"
> >>+#else
> >>+# define BALIGN_8 ".balign 8 \n\t"
> >>+# define BALIGN_16 ".balign 16 \n\t"
> >>+#endif
> >>
> >>Is this really so ugly as to be a scar upon the codebase? If not,
> >>then what in the linked patch is truly so objectionable?
> >>
> >>
> >
> >It's ugly because it makes an assumption about a platform which may or
> >may not be true in the future.
> >
>
> Maybe in the future if a patch is rejected for "ugly" it might be better
> to explain what is ugly about it?
where did i reject a patch with just "ugly" ?
and if some comment to a patch is unclear, then why not simply ask what the
person who wrote the comment meant? "why is that ugly, how could it be changed
so its not ugly"
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is
More information about the ffmpeg-devel
mailing list