*/
void MailboxImpl::push(CommImplPtr comm)
{
- comm->mbox = this;
+ comm->set_mailbox(this);
this->comm_queue_.push_back(std::move(comm));
}
*/
void MailboxImpl::remove(const CommImplPtr& comm)
{
- xbt_assert(comm->mbox == this, "Comm %p is in mailbox %s, not mailbox %s", comm.get(),
- (comm->mbox ? comm->mbox->get_cname() : "(null)"), this->get_cname());
- comm->mbox = nullptr;
+ xbt_assert(comm->get_mailbox() == this, "Comm %p is in mailbox %s, not mailbox %s", comm.get(),
+ (comm->get_mailbox() ? comm->get_mailbox()->get_cname() : "(null)"), this->get_cname());
+
+ comm->set_mailbox(nullptr);
for (auto it = this->comm_queue_.begin(); it != this->comm_queue_.end(); it++)
if (*it == comm) {
this->comm_queue_.erase(it);
xbt_die("Comm %p not found in mailbox %s", comm.get(), this->get_cname());
}
-CommImplPtr MailboxImpl::iprobe(int type, int (*match_fun)(void*, void*, CommImpl*), void* data)
+CommImplPtr MailboxImpl::iprobe(int type, bool (*match_fun)(void*, void*, CommImpl*), void* data)
{
XBT_DEBUG("iprobe from %p %p", this, &comm_queue_);
CommImplPtr this_comm;
CommImpl::Type smx_type;
if (type == 1) {
- this_comm = CommImplPtr(new CommImpl(CommImpl::Type::SEND));
+ this_comm = CommImplPtr(new CommImpl());
+ this_comm->set_type(CommImpl::Type::SEND);
smx_type = CommImpl::Type::RECEIVE;
} else {
- this_comm = CommImplPtr(new CommImpl(CommImpl::Type::RECEIVE));
+ this_comm = CommImplPtr(new CommImpl());
+ this_comm->set_type(CommImpl::Type::RECEIVE);
smx_type = CommImpl::Type::SEND;
}
CommImplPtr other_comm = nullptr;
* @param remove_matching whether or not to clean the found object from the queue
* @return The communication activity if found, nullptr otherwise
*/
-CommImplPtr MailboxImpl::find_matching_comm(CommImpl::Type type, int (*match_fun)(void*, void*, CommImpl*),
+CommImplPtr MailboxImpl::find_matching_comm(CommImpl::Type type, bool (*match_fun)(void*, void*, CommImpl*),
void* this_user_data, const CommImplPtr& my_synchro, bool done,
bool remove_matching)
{
auto& comm_queue = done ? done_comm_queue_ : comm_queue_;
for (auto it = comm_queue.begin(); it != comm_queue.end(); it++) {
- CommImplPtr& comm = *it;
+ const CommImplPtr& comm = *it;
- if (comm->type == CommImpl::Type::SEND) {
+ if (comm->type_ == CommImpl::Type::SEND) {
other_user_data = comm->src_data_;
- } else if (comm->type == CommImpl::Type::RECEIVE) {
+ } else if (comm->type_ == CommImpl::Type::RECEIVE) {
other_user_data = comm->dst_data_;
}
- if (comm->type == type && (match_fun == nullptr || match_fun(this_user_data, other_user_data, comm.get())) &&
+ if (comm->type_ == type && (match_fun == nullptr || match_fun(this_user_data, other_user_data, comm.get())) &&
(not comm->match_fun || comm->match_fun(other_user_data, this_user_data, my_synchro.get()))) {
XBT_DEBUG("Found a matching communication synchro %p", comm.get());
#if SIMGRID_HAVE_MC
- comm->mbox_cpy = comm->mbox;
+ comm->mbox_cpy = comm->get_mailbox();
#endif
- comm->mbox = nullptr;
+ comm->set_mailbox(nullptr);
CommImplPtr comm_cpy = comm;
if (remove_matching)
comm_queue.erase(it);
}
XBT_DEBUG("Sorry, communication synchro %p does not match our needs:"
" its type is %d but we are looking for a comm of type %d (or maybe the filtering didn't match)",
- comm.get(), (int)comm->type, (int)type);
+ comm.get(), (int)comm->type_, (int)type);
}
XBT_DEBUG("No matching communication synchro found");
return nullptr;