Re: [FFmpeg-devel] [PATCH] avutil/cpu: remove the |checked| static variable

2016-11-23 Thread Michael Niedermayer
On Wed, Nov 23, 2016 at 11:21:56AM -0800, Wan-Teh Chang wrote: > Remove the |checked| variable because the invalid value of -1 for > |flags| can be used to indicate the same condition. Also rename |flags| > to |cpu_flags| because there are a local variable and a function > parameter named |flags| i

[FFmpeg-devel] [PATCH] avutil/cpu: remove the |checked| static variable

2016-11-23 Thread Wan-Teh Chang
Remove the |checked| variable because the invalid value of -1 for |flags| can be used to indicate the same condition. Also rename |flags| to |cpu_flags| because there are a local variable and a function parameter named |flags| in the same file. Add a test program libavutil/tests/cpu_init.c to chec