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

2008-07-08 Thread Ross Finlayson
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

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

2008-07-08 Thread Erik Hovland
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-all

Re: [Live-devel] [patch] make sure string storage has enough room for the null char

2008-07-08 Thread Ross Finlayson
This patch is the one I am least comfortable with. I have no idea if the strings that are being created require a null char to terminate them. But to stimulate discussion I am posting the patches that do make sure there is room for the null char in a few strings where it seems it was no accounted

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

2008-07-08 Thread Ross Finlayson
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. --

Re: [Live-devel] [patch] use const_cast instead of C casting

2008-07-08 Thread Erik Hovland
On Tue, Jul 08, 2008 at 09:24:39AM -0700, Ross Finlayson wrote: > This is one patch that I won't be adding, because I want the code to > work even with older compilers that might not understand "const_char" ( > or even templates). Not a problem. Thanks for the quick feedback. E -- Erik Hovla

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

2008-07-08 Thread Erik Hovland
On Tue, Jul 08, 2008 at 09:13:01AM -0700, Erik Hovland wrote: > A lot of const char* = "" calls means that g++ complains about how that > feature is deprecated. As well as some casting and some changing of > function declarations (socketErr). This patch has been checked for > memory leaks since it

Re: [Live-devel] [patch] use const_cast instead of C casting

2008-07-08 Thread Ross Finlayson
This is one patch that I won't be adding, because I want the code to work even with older compilers that might not understand "const_char" ( or even templates). Ross. -- Sent from my jesusPhone On Jul 8, 2008, at 9:03 AM, Erik Hovland <[EMAIL PROTECTED]> wrote: In C++ it is

[Live-devel] [patch] make sure string storage has enough room for the null char

2008-07-08 Thread Erik Hovland
This patch is the one I am least comfortable with. I have no idea if the strings that are being created require a null char to terminate them. But to stimulate discussion I am posting the patches that do make sure there is room for the null char in a few strings where it seems it was no accounted f

[Live-devel] [patch] initialized member variables in ctor

2008-07-08 Thread Erik Hovland
OK, now I am really starting to run out of issues. How do I know? Well, I am starting to produce the member variable initialization in constructor patches. These defects are just because it is good C++ practice to initialize member variables in the ctor of the class to make sure that these member v

[Live-devel] [patch] use const_cast instead of C casting

2008-07-08 Thread Erik Hovland
In C++ it is preferred to control constness by using const_cast instead of the C casting syntax. This patch does that in a couple of places. E --- liveMedia/QuickTimeGenericRTPSource.cpp |2 +- liveMedia/SimpleRTPSource.cpp |2 +- 2 files changed, 2 insertions(+), 2 deletions(

Re: [Live-devel] MPEG4 hinted file for RTP

2008-07-08 Thread Ross Finlayson
I wanted to ask one question regarding RTP. I was able to get a file of a MPEG4 video already hinted for RTP. I wanted to ask how could I stream this file. You need to use Apple's "Darwin Streaming Server". Our RTSP server implementation cannot currently stream such a file. (Note, once agai

[Live-devel] MPEG4 hinted file for RTP

2008-07-08 Thread Fernando ReƔtegui del Aguila
Hello, I wanted to ask one question regarding RTP. I was able to get a file of a MPEG4 video already hinted for RTP. I wanted to ask how could I stream this file. Fernando. ___ live-devel mailing list live-devel@lists.live555.com http://lists.live555.co

Re: [Live-devel] Asking about live555 Media server

2008-07-08 Thread Ross Finlayson
Because I can't type the address rstp:// into the web browser You can't play "rtsp://" URLs from (most) web browsers. Instead, you need a separate RTSP client application, like VLC , also don't have any server keep media files, so it means I must put a file w

Re: [Live-devel] About RTSP over UDP behind firewall.

2008-07-08 Thread Ross Finlayson
So my question is that whether live55 have any plan to support similar mechanism with RTP over UDP to bypass the firewall? Yes, the IETF is currently working to standardize such a mechanism, and I plan on supporting this in our software once the standardization is complete (or close to comp

[Live-devel] Asking about live555 Media server

2008-07-08 Thread Hannah
Hi, I have some questions to ask. Hope someone answer me something. 1. I've built the live555MediaServer. It says that to stream from this server, simply type the URL like : rtsp://172.18.75.202:8554/ where is a file present in the current directory. Because I can't type the address rstp:// into