Is this a bug of PlayTaskProducer?

[Wuqiqi123]

Just like the code in PlayTaskProducer::Stop, if the is_stopped_ is false(running), stop function will just return directly.

apollo/cyber/tools/cyber_recorder/player/play_task_producer.cc

Lines 95 to 103 in 22cb4d8

	void PlayTaskProducer::Stop() {
	if (!is_stopped_.exchange(true)) {
	return;
	}
	if (produce_th_ != nullptr && produce_th_->joinable()) {
	produce_th_->join();
	produce_th_ = nullptr;
	}
	}

Is this changes right?

-   if (!is_stopped_.exchange(true)) { 
+   if (is_stopped_.exchange(true)) { 
     return; 
   } 

Also the PlayTaskConsume::Stop looks right here.

apollo/cyber/tools/cyber_recorder/player/play_task_consumer.cc

Lines 58 to 70 in 22cb4d8

	void PlayTaskConsumer::Stop() {
	if (is_stopped_.exchange(true)) {
	return;
	}
	if (consume_th_ != nullptr && consume_th_->joinable()) {
	consume_th_->join();
	consume_th_ = nullptr;
	}
	// reset value for next use consumer
	base_msg_play_time_ns_ = 0;
	base_msg_real_time_ns_ = 0;
	last_played_msg_real_time_ns_ = 0;
	}