On 10/06/15 15:29, Vittorio Giovara wrote: > Having this macro in an header only facilitates the use of such header. > The code increase is minimal and files have one less dependency > on mpegvideo.h. > --- > libavcodec/flvenc.c | 7 ++++++- > libavcodec/h261enc.c | 7 ++++++- > libavcodec/mpegvideo.h | 8 -------- > libavcodec/mpegvideo_enc.c | 21 ++++++++++++++++++--- > libavcodec/rv10enc.c | 7 ++++++- > libavcodec/rv20enc.c | 7 ++++++- > 6 files changed, 42 insertions(+), 15 deletions(-) >
The dependency isn't the same if they are using the same options (thus the same struct?)? The patch it is fine since moves one step further to decouple codecs from mpegvideoenc... lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
