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

[Ramiro Ribeiro Polla]

Michael Niedermayer wrote:
> 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
>
>   

Applied.