[MPlayer-dev-eng] [PATCH] Optional VIDIX PCI devices name database

Benjamin Zores ben at geexbox.org
Tue Dec 4 19:11:10 CET 2007


Diego Biurrun a écrit :
> On Mon, Dec 03, 2007 at 08:01:03PM +0100, Benjamin Zores wrote:
>> Diego Biurrun a écrit :
>>> On Mon, Dec 03, 2007 at 12:42:19PM +0100, Benjamin Zores wrote:
>>>> On Dec 3, 2007 11:57 AM, Diego Biurrun <diego at biurrun.de> wrote:
>>>>> On Sun, Dec 02, 2007 at 06:56:20PM +0100, Benjamin Zores wrote:
>>>>>> Opinions ?
>>>>>>
>>>>>> --- vidix/Makefile    (revision 25211)
>>>>>> +++ vidix/Makefile    (working copy)
>>>>>> @@ -29,6 +29,12 @@
>>>>>>  OBJS_MPLAYER += rage128_vid.o
>>>>>>  endif
>>>>>>
>>>>>> +ifeq ($(VIDIX_PCIDB), yes)
>>>>>> +PCI_DB = 1
>>>>>> +else
>>>>>> +PCI_DB = 0
>>>>>> +endif
>>>>> This is completely redundant.  Just set PCI_DB directly in configure.
>>>> One is string, other is integer.
>>>> I have absolutely no knowledge at all about awk, but does "if(yes)" do
>>>> the same than "if(1)" ?
>>> I have no idea about awk, but nothing stops you from setting PCI_DB in
>>> configure.
>> Better like this way ?
> 
> Yes.  No further objections from my side apart from the following
> spelling nit:

Applied with proposed suggestions.

Ben



More information about the MPlayer-dev-eng mailing list