EventLoopThread::startLoop()中为什么需要创建局部变量loop？是否可以返回loop_?

[lamb007]

EventLoop* EventLoopThread::startLoop()
{
  assert(!thread_.started());
  thread_.start();

  EventLoop* loop = NULL;
  {
    MutexLockGuard lock(mutex_);
    while (loop_ == NULL)
    {
      cond_.wait();
    }
    loop = loop_;
  }

  return loop;
}

void EventLoopThread::threadFunc()
{
  EventLoop loop;

  if (callback_)
  {
    callback_(&loop);
  }

  {
    MutexLockGuard lock(mutex_);
    loop_ = &loop;
    cond_.notify();
  }

  loop.loop();
  //assert(exiting_);
  MutexLockGuard lock(mutex_);
  loop_ = NULL;
}

EventLoopThread::startLoop()是否可以实现为下面的方式？

EventLoop* EventLoopThread::startLoop() {
  thread_.start();
  std::unique_lock lock(mutex_);
  cv_.wait(lock, [this]() { return loop_ != nullptr; });
  return loop_;
}

望赐教，谢谢

[leabol]

应该是可以, 原代码那样写应该是为了规范代码,明确语义吧. 所有对成员变量的访问，如果该变量可能被多线程访问，就必须在锁的保护下进行。