On Sat, Jan 12, 2008 at 12:48:07PM +0000, Udo Giacomozzi wrote:
> + /// Set the 'querystring' member of this URL to a new value
> + ///
> + void set_querystring(std::string value) { _querystring = value; }
Take a const std::string& value there, or the value would be copied
twice (once to pass as parameter, once to copy to _queristring).
Also, should you check for _querystring being non-empty ?
I mean, you may want to *append* values, not necessarely
destroy any older one.
--strk;
_______________________________________________
Gnash-commit mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/gnash-commit
