Måns Rullgård <[email protected]> writes: > Luca Barbato <[email protected]> writes: > >> On 06/16/2012 06:33 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sat, Jun 16, 2012 at 2:21 AM, Luca Barbato <[email protected]> wrote: >>>> On 06/16/2012 03:02 AM, Ronald S. Bultje wrote: >>>>> I wish to support a compiler that does not support c99. Shall we work >>>>> together and make that happen or shall we just cry like little zealous >>>>> babies while the real world passes by? >>>> >>>> Reality is that either you add 10 lines in the preprocessor that is >>>> already running over the libav sources or you change 20 lines in 7 files. >>>> >>>> Reason suggests to keep the 10 lines in the preprocessor. >>> >>> It's not 10 lines. >> >> sed -e "s:\\* *\(restrict[ )]\):*__\1:g" >> >> seems to produce what is the expected result, >> >>> I've defended the patch enough, I ask you to please >>> accept it as-is or suggest another patch in Libav that gives the same >>> result. >> >> The patch itself is saner than the status quo (since it is broken as you >> stated). >> >> So either we drop the rename in configure (and then we use a sed line >> like the one I wrote) or we accept the patch. > > None of the compilers we currently support need the renaming,
In fact, no compiler I have at hand, even ones we don't support, need it. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
