[MPlayer-dev-eng] [PATCH] configure: enhancement for runtime cpu detection and --target support under x86
Diego Biurrun
diego at biurrun.de
Wed Dec 13 12:11:49 CET 2006
On Wed, Dec 13, 2006 at 03:51:03PM +0800, Zuxy Meng wrote:
>
> 2006/12/13, Diego Biurrun <diego at biurrun.de>:
> >On Wed, Dec 13, 2006 at 09:58:38AM +0800, Zuxy Meng wrote:
> >> Well, I don't guess so. If u add a pair of brackets around some old
> >> statements in C you're expected to reindent those statements. Similar
> >> in this case.
> >
> >Time to read DOCS/tech/patches.txt and DOCS/tech/svn-howto.txt. Such a
> >change should go ina separate patch.
>
> Well let me elaborate: if the original code looks like
>
> memcpy(dst, src, len);
>
> and I wish to change it to
>
> if (dst)
> memcpy(dst, src, len);
>
> Do I have to first do
>
> +if (dst)
> memcpy(dst, src, len);
>
> and then
>
> if(dst)
> -memcpy(dst, src, len);
> + memcpy(dst, src, len);
>
> ???
For just one line, no. If you put 'if' (or similar) around a large
number of lines, yes.
Diego
More information about the MPlayer-dev-eng
mailing list