[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