Pass one shot constructor movable parameters by value

[pepone]

The generated C++ one-shoot constructor for a class looks like this:

MessageEvent(
    long long int timestamp,
    const ::std::string& name,
    const ::std::string& message) :
    Ice::ValueHelper<MessageEvent, ChatRoomEvent>(timestamp, name),
    message(message) {
}

It would be better to pass std::string by value and use std::move which will allow avoiding copies when the parameter is a lreference