Ensure a QByteArray owns its memory (QByteArray::fromRawData)

litb_34509 0
Viewed 2549

Say we have a function store

void store(const QByteArray& data);

This function's job is to take data and store it away. Unfortunately, it is not safe to do that if the argument was created with QByteArray::fromRawData(ptr, size), because it and all its copies require that ptr remains valid.

Therefore store has no option to forbid its callers to pass in such an array, to treat data as if it was a const char* on stereoids or to force a deep-copy with detach. All of this is not satisfying and especially the latter hurts performance because if data was COW-copied before being passed to store, we will be doing an unnecessary deep-copy.

QByteArray has a private function nulTerminated whose implementation seems to do just what I want: If it doesn't own the memory, it deep copies. And if it owns the memory, it does nothing but return *this.

Two questions really

  • Is there a work-around using public facilities?

  • The Qt docs mention that ptr must only be alive for the lifetime of the return value and any copies of it. If you say .right(.size()), it would seem this is not a copy, so Qt would need to make a deep copy according to the docs. But does it really do so?

qbytearray qt c++
2 Answers

this sounds more like a conceptual problem, rather than technical.

store() tries to take absolute ownership, but "taking ownership" on a QByteArray does not make much sense conceptually. A copy might share its memory with another, or it might not, or it might be created from raw data, so there's a hidden ownership juggle going on under the hood and the whole class is designed to hide this from you. The whole implicit sharing thing doesn't mesh well with modern explicit ownership transfer paradigm in general, and QByteArray complicates matters further by providing the raw data facility.

If possible, accepting a non-implicitly-shared container that holds absolute ownership over their content (such as std::string, which is easily generated with QByteArray::toStdString()) by rvalue, forcing callers to copy, would be the best option.

user_2947630
Related