Diego Biurrun <di...@biurrun.de> writes:

> On Tue, Apr 17, 2012 at 11:46:47AM +0100, Måns Rullgård wrote:
>> Diego Biurrun <di...@biurrun.de> writes:
>> > --- a/libavcodec/dv_tablegen.h
>> > +++ b/libavcodec/dv_tablegen.h
>> > @@ -25,7 +25,6 @@
>> >
>> > -#include "libavutil/attributes.h"
>> >  #include "dv_vlc_data.h"
>> >
>> > @@ -48,7 +47,7 @@ typedef struct dv_vlc_pair {
>> >
>> > -static void av_unused dv_vlc_map_tableinit(void)
>> > +static void dv_vlc_map_tableinit(void)
>> >  {
>> >      int i, j;
>> >      for (i = 0; i < NB_DV_VLC - 1; i++) {
>> 
>> Why did you add it in the first place?
>
> I honestly cannot tell for sure by now, but I suspect it was before a
> buggy version of the split that #included dv_tablegen.h in more than
> one place.

Well, it ought not be needed, so patch OK.

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to