[MPlayer-cvslog] r22623 - in trunk/libmpdemux: Makefile asfheader.c aviheader.c aviprint.c demux_aac.c demux_asf.c demux_audio.c demux_avi.c demux_avs.c demux_demuxers.c demux_film.c demux_fli.c demux_gif.c demux_lavf.c demux_lmlm4.c demux_mf.c d

Diego Biurrun diego at biurrun.de
Thu Mar 15 23:28:46 CET 2007


On Thu, Mar 15, 2007 at 08:51:19PM +0200, Ivan Kalvachev wrote:
> 2007/3/15, diego <subversion at mplayerhq.hu>:
> >
> >Log:
> >Add explicit location for headers from the stream/ directory.
> 
> Would you explain why this is improvement?
> I don't think that one line^Wword  less in makefile and hard-coding
> include file in 54 files is worth it.

Sometimes we include the path name, sometimes we don't.  Now it's
consistent and more obvious where the header actually resides.  Plus, no
problems with headers that have the same name.

> Not to say that it makes moving stuff around harder.

sed is your friend.

Diego




More information about the MPlayer-cvslog mailing list