gnash-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnash-commit] gnash ChangeLog libamf/amf.cpp server/vm/Action...


From: Rob Savoye
Subject: Re: [Gnash-commit] gnash ChangeLog libamf/amf.cpp server/vm/Action...
Date: Sun, 18 May 2008 18:23:38 +0200
User-agent: Thunderbird 2.0.0.9 (X11/20071031)

strk wrote:

But really is just an optimization, as long as you're modifying
that code anyway what is it worth ?

My point exactly. :-) it's a temporary debug message, and I've stated several times it'll get fixed while I continue working on the AMF code as I create the client side of RTMP for Gnash. But if you want to fix the valgrind warning, then type cast tmpptr to name, rather than creating a std::string. As the ASCII string of the property name is in tmpptr, allocating a string is a performance hit. ()although a small one)

        - rob -




reply via email to

[Prev in Thread] Current Thread [Next in Thread]