On Sat, Apr 05, 2014 at 02:16:48PM +0200, Janne Grunau wrote: > --- /dev/null > +++ b/libavcodec/aarch64/videodsp.S > @@ -0,0 +1,28 @@ > +// > +// This file is part of Libav. > +// > +// Libav is free software; you can redistribute it and/or > +// modify it under the terms of the GNU Lesser General Public > +// License as published by the Free Software Foundation; either > +// version 2.1 of the License, or (at your option) any later version. > +// > +// Libav is distributed in the hope that it will be useful, > +// but WITHOUT ANY WARRANTY; without even the implied warranty of > +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > +// Lesser General Public License for more details. > +// > +// You should have received a copy of the GNU Lesser General Public > +// License along with Libav; if not, write to the Free Software > +// Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 > USA > + > +#include "config.h" > +#include "libavutil/aarch64/asm.S"
This looks strange w/o multiline comments, like we use everywhere else. > --- /dev/null > +++ b/libavcodec/aarch64/videodsp_init.c > @@ -0,0 +1,32 @@ > + > +#include "libavutil/aarch64/cpu.h" > +#include "libavutil/attributes.h" > +#include "libavutil/cpu.h" nit: Move aarch64 #include below the others. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
