On 05.10.2010 08:28, Jan Hoogenraad wrote: > The file dmx.h declares dmx_pes_type_t which is ONLY used in this header > file. > It is easy to make mistakes, as its member DMX_PES_OTHER resembles a > similar symbol in demux.h from enum dmx_ts_pes: DMX_TS_PES_OTHER > > One problem was found because of Hans' compiler upgrade. > > I think it would be safer (because it is easy to make mistakes) to > remove dmx_pes_type_t from dmx.h. The file dmx.h defines the interface for userspace applications, hence dmx_pes_type_t must not be removed. Instead, removing enum dmx_ts_pes from demux.h seems to be useful to me. Regards, Andreas -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html