Francesco Lavra <francescolavra at interfree.it> writes: >> > +#include "riff.h" >> >> Why? Rest of patch #1 is OK. > > Because AVCodecTag is defined there. This has bothered me for some time. IMO we should move that struct somewhere else. It is in no way specific to the RIFF formats. -- M?ns Rullg?rd mans at mansr.com