Seems like (more or less) an even trade-off to me as to what we had before.
typing:
auto b = bundle->compositor_acquire(...);
b->buffer()->methods();
saves on having to update the Temporary*Buffer adapter objects, but is more awkward to type, compared to:
auto b = bundle->compositor_acquire(...);
b->methods();
84 +mc::BufferHandle::BufferHandle(BufferBundle* bundle,
We used to preserve ownership of the BufferBundle in the temporary buffers, seems like something that could cause problems around the client's destruction if we don't have BufferBundle ownership, and are trying to return in the destructor of BufferHandle.
We already have a "native handle", might be over-using the word "Handle" in mc::BufferHandle. (I'm mostly just trying to avoid writing: mg::BufferHandle handle{...}; handle->buffer()->native_buffer_handle() )
Seems like (more or less) an even trade-off to me as to what we had before. >compositor_ acquire( ...); )->methods( ); >compositor_ acquire( ...);
typing:
auto b = bundle-
b->buffer(
saves on having to update the Temporary*Buffer adapter objects, but is more awkward to type, compared to:
auto b = bundle-
b->methods();
84 +mc::BufferHand le::BufferHandl e(BufferBundle* bundle,
We used to preserve ownership of the BufferBundle in the temporary buffers, seems like something that could cause problems around the client's destruction if we don't have BufferBundle ownership, and are trying to return in the destructor of BufferHandle.
169 + explicit CompositorBuffe rHandle( BufferBundle* bundle, andle(BufferBun dle* bundle,
177 + explicit SnapshotBufferH
spacing
161 + std::shared_ ptr<graphics: :Buffer> buffer); ptr<graphics: :Buffer> buffer); ptr<graphics: :Buffer> buffer);
170 + std::shared_
178 + std::shared_
const&
We already have a "native handle", might be over-using the word "Handle" in mc::BufferHandle. (I'm mostly just trying to avoid writing: mg::BufferHandle handle{...}; handle- >buffer( )->native_ buffer_ handle( ) )
257 + std::shared_ ptr<mc: :BufferHandle> const acquired_buffer = shared< mc::SnapshotBuf ferHandle> (this, buffer_ for(current_ compositor_ buffer, buffers)); shared< mc::SnapshotBuf ferHandle( ...)
258 + std::make_
259 +
260 + return acquired_buffer;
could just:
return std::make_