diff options
author | Radoslaw Zarzynski <rzarzyns@redhat.com> | 2021-07-19 15:55:18 +0200 |
---|---|---|
committer | Radoslaw Zarzynski <rzarzyns@redhat.com> | 2021-07-20 16:29:34 +0200 |
commit | edf3bbc004409098b9bb3ec7a369a99b51cb131a (patch) | |
tree | 4030f5a5eeff14bd97bd4965a2907a3dfd6f75ed /systemd/ceph-osd@.service.in | |
parent | Merge pull request #42334 from tchaikov/wip-crimson-logging (diff) | |
download | ceph-edf3bbc004409098b9bb3ec7a369a99b51cb131a.tar.xz ceph-edf3bbc004409098b9bb3ec7a369a99b51cb131a.zip |
common/bl: c_str() examines length instead of no. of bptrs.
This approach has 2 adventages:
* it takes into consideration the specical case of a bufferlist
that carries solely empty bptrs while
* allowing compilers to optimize-out the load-and-check of `_len`
we already do in `rebuild()` which is called by `c_str()`.
```cpp
void buffer::list::rebuild()
{
if (_len == 0) {
_carriage = &always_empty_bptr;
_buffers.clear_and_dispose();
_num = 0;
return;
}
// ...
}
```
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Diffstat (limited to 'systemd/ceph-osd@.service.in')
0 files changed, 0 insertions, 0 deletions