[Ffmpeg-devel] [PATCH] '/nop' illegal for old versions of GAS

Michael Niedermayer michaelni
Mon Aug 7 12:29:05 CEST 2006


Hi
On Mon, Aug 07, 2006 at 12:00:48PM +0200, Michael Niedermayer wrote:
[...]
> > 	If modifying the macro to remove the rounding ability
> > is OK, then this all becomes easy - just comment it out:
> > 
> > -        #rounder ", %%mm4               \n\t"\
> > +        /* #rounder ", %%mm4            \n\t" Add a rounding constant 
> > */\
> 
> sorry no, i summarize it again
> NOP must be a macro defined by #define
> the output of the code must not change
> the code must be clean, the patch must conform to whats described in our
> docs
> theres nothing tricky with that its a plain and simple change
> 
> you dont seem to know how the c preprocessor works, and you dont seem to
> be willing to learn it / read the manual thats fine but then you dont
> qualify as contributor, as knowledge of the language or at least will to
> read the manuals is required to contribute to a project which is written
> in that language, again,
> i dont need mac intel support, if someone wants it he has to do the work
> 
> also note that if the trolling about macintel continues
> (trolling here means repeatly sending patches and proposing things which ive
> said already are not ok) then ill ban thouse who send such spam, as it
> confuses others into thinking that someone would be working on macintel
> support while we are actually just playing with a troll who never intended
> to send a useable patch but instead is trying to cause as much work for us
> as possible, again the patch submitter must do the work not we, this is true
> for every patch. alternatives are not possible for a project of the size of
> ffmpeg, we simply cannot rewrite everything people submit

after reading my own post again it seems ive been a little more harsh then
intended, so sorry but what you proposed is not ok,

just replacing 
code like:
ROW_IDCT(  64(%0), 72(%0), 80(%0), 88(%0), 64(%1), paddd (%2), 11)
with:
ROW_IDCT(  64(%0), 72(%0), 80(%0), 88(%0), 64(%1), "paddd (%2)", 11)

together with the other changes i proposed will work
and is IMO nothing special but just normal use of the pre processor

[...]
-- 
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