[FFmpeg-devel] [PATCH] libavdevice: make device_try_init static
Luca Abeni
lucabe72
Mon Jan 4 20:40:06 CET 2010
On 04/01/10 19:44, Diego Biurrun wrote:
> On Mon, Jan 04, 2010 at 06:32:16PM +0100, Luca Abeni wrote:
>>
>> On 03/01/10 12:55, Diego Biurrun wrote:
>>> On Sun, Jan 03, 2010 at 11:45:28AM +0100, Luca Abeni wrote:
>>>> On 02/01/10 10:44, Reinhard Tartler wrote:
>>>>> $subj.
>>>>
>>>> Patch applied, thanks! BTW, do you have some automated scripts or tools
>>>> for performing this kind of checks? It could be interesting to integrate
>>>> them in FATE, or similar...
>>>
>>> -Wmissing-prototypes
>>> -Wmissing-declarations
>>
>> Thanks for the information, this is very useful!
>> What about adding those switches to ffmpeg compilation? I do not know
>> the build system very much, but I am wondering if the attached patch is
>> acceptable.
>
> I played with it some time ago. However, I still wonder what the
> difference between both is. Enlightenment welcome.
Eh... I was going to ask a similar question ;-)
I guess I need to read the C standard searching for the difference
between function declarations and prototypes...
>> I tested it, and it even produced some new warnings ;-)
>
> Did you fix them? :)
I looked at them, and some of the warning looks trivial to fix (missing
#includes, missing "static", etc...). I'll send patches (or directly
apply the fixes, as people prefer) as soon as I'll have some time.
Thanks,
Luca
More information about the ffmpeg-devel
mailing list