[Ffmpeg-devel] [RFC] ffmpeg-windows mailinglist?
Chris Forsythe
chris
Wed Aug 2 07:28:13 CEST 2006
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?
Chris
More information about the ffmpeg-devel
mailing list