Graham Palmer

Results 57 comments of Graham Palmer

@elBoberido I'm back from vacation whenever you want to continue discussing this. If you'd like, we can add an agenda item to one of the next two developer meetups. Though...

I've made some changes to the design to remove the subscriber segment requesting feature and change how segments are requested to be mapped in the runtime. I'll go through again...

Took another look. There doesn't seem to be any issue with removing a chunk when we know its index in the `UsedChunkList`. That would essentially just skip to this line...

Yes so, after my first quick pass I'm thinking we can update the API's in the following way to achieve the goal of eliminating this linear search: 1. For `UsedChunkList::insert`,...

> But keep in mind, the UsedChunkList is a lock-free construct This isn't relevant as the data structure is not meant to be used concurrently. It is even explicitly NOT...

As for our setup. 1. There are just a few subscribers per topic, usually, yes. But there could be more depending on the use case. 2. The problem described here...

No worries 🙂 I just took a closer look and there is a slight complication to the approach I outlined. In true forward list fashion, list removal does unfortunately require...

Note that this information basically gets immediately plumbed into the deleter of a `unique_ptr` [here](https://github.com/gpalmer-latai/iceoryx/blob/e97d209e13c36d71237c99e0246310b8029f8f26/iceoryx_posh/include/iceoryx_posh/internal/popo/subscriber_impl.inl#L49)

Ah, but the untyped subscriber doesn't so neatly encode a destructor: https://github.com/gpalmer-latai/iceoryx/blob/e97d209e13c36d71237c99e0246310b8029f8f26/iceoryx_posh/include/iceoryx_posh/internal/popo/untyped_subscriber_impl.inl#L54, so we'd have to store this information elsewhere...

> @gpalmer-latai Instead of using some kind of list, couldn't you fill the `m_listData` with `optional`s and set them to `nullopt` when it is no longer set? The performance hit...