Re: [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
2007/3/15, diego <subversion@mplayerhq.hu>:
Author: diego Date: Thu Mar 15 19:36:36 2007 New Revision: 22623
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. Not to say that it makes moving stuff around harder.
On Thu, Mar 15, 2007 at 08:51:19PM +0200, Ivan Kalvachev wrote:
2007/3/15, diego <subversion@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
participants (2)
-
Diego Biurrun -
Ivan Kalvachev