[FFmpeg-devel] [PATCH] imlib2's rgb.txt

[Michael Niedermayer]

Hi

On Thu, Jun 14, 2007 at 08:29:17PM -0300, Ramiro Ribeiro Polla wrote:
> Michael Niedermayer wrote:
> >Hi
> >
> >On Mon, Jun 11, 2007 at 01:28:31PM -0300, Ramiro Ribeiro Polla wrote:
> >  
> >>Ramiro Ribeiro Polla wrote:
> >>    
> >>>Michael Niedermayer wrote:
> >>>  
> >>>      
> >>>>Hi
> >>>>
> >>>>On Mon, Jun 11, 2007 at 09:06:34AM +0200, V?ctor Paesa wrote:
> >>>>  
> >>>>    
> >>>>        
> >>>>>Hi,
> >>>>>
> >>>>>    
> >>>>>      
> >>>>>          
> >>>>>>Attached patch adds another location for the imlib2 vhook to search 
> >>>>>>for
> >>>>>>rgb.txt.
> >>>>>>It's a relative path instead of an absolute path, good for Windows,
> >>>>>>where X11 isn't installed.
> >>>>>>
> >>>>>>Even though vhooks should go away soon, it's nice for the next filter
> >>>>>>system to support this too, so this patch is also informative for
> >>>>>>whoever writes the next imlib2 filter.
> >>>>>>      
> >>>>>>        
> >>>>>>            
> >>>>>The patch is also useful for POSIX systems that don't have X11 
> >>>>>installed, or
> >>>>>have it installed in unusual directories.
> >>>>>    
> >>>>>      
> >>>>>          
> >>>>would it not make more sense to allow the user to specifiy the file 
> >>>>path?
> >>>>
> >>>>  
> >>>>    
> >>>>        
> >>>Yes, it would.
> >>>Also, it would make sense to add hex value support like drawtext.
> >>>
> >>>How about this:
> >>>- add -rgb parameter to specify rgb.txt file
> >>>      
> >
> >probably ok
> >
> >
> >  
> >>>- remove the 2 default paths for rgb.txt (ok?)
> >>>      
> >
> >i think some default should be kept somewhere
> >
> >
> >  
> 
> Is attached patch ok? (then another commit to reindent)

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070615/dad4a8ff/attachment.pgp>