[Live-devel] [patch] fix or suppress compiler warnings

Ross Finlayson finlayson at live555.com
Tue Jul 8 21:56:45 PDT 2008


>On Tue, Jul 08, 2008 at 05:31:59PM -0700, Ross Finlayson wrote:
>>>  This patch has been checked for
>>>   > memory leaks since it almost always heap allocates a duplicate string
>>>   > instead of the = "".
>>
>>  I see no reason not to continue to use static string constants. Copying
>>  them into heap-allocated memory instead (using "strDup()") is 
>>  inefficient and pointless.
>
>Understood. I'll submit a new patch that doesn't fix that. When gcc 4.4
>and the feature is truly deprecated I can rework it.

Perhaps what your compiler is really complaining about is the fact 
that these string constants are declared
	char*
instead of
	char const*
or
	char const* const

Perhaps that's all that's needed to satisfy it?
-- 

Ross Finlayson
Live Networks, Inc.
http://www.live555.com/


More information about the live-devel mailing list