Re: [libav-devel] [PATCH] options_table: Remove a now unnecessary include of config.h

2016-11-03 Thread Martin Storsjö
On Thu, 3 Nov 2016, Martin Storsjö wrote: The include of config.h was added in 2012 in 1d9c2dc8, due to the use of CONFIG_SHOW_ENCODER ifdefs within options_table.h. CONFIG_SNOW_ENCODER, of course. Amended locally. // Martin ___ libav-devel mailing

Re: [libav-devel] [PATCH] options_table: Remove a now unnecessary include of config.h

2016-11-03 Thread Luca Barbato
On 03/11/2016 09:58, Martin Storsjö wrote: > The include of config.h was added in 2012 in 1d9c2dc8, due to > the use of CONFIG_SHOW_ENCODER ifdefs within options_table.h. > When the snow codec was dropped later (in a0c5917f8 in 2013), > this include no longer served any purpose. Ok with

[libav-devel] [PATCH] options_table: Remove a now unnecessary include of config.h

2016-11-03 Thread Martin Storsjö
The include of config.h was added in 2012 in 1d9c2dc8, due to the use of CONFIG_SHOW_ENCODER ifdefs within options_table.h. When the snow codec was dropped later (in a0c5917f8 in 2013), this include no longer served any purpose. options_table.h is included in builds for the host as well, when