Skip to content

Commit dd8c4e4

Browse files
[ISSUE #6316]Remove unnecessary .as_str() call in DefaultMQProducer#start (#6318)
- Replaced .clone().as_str() with cleaner approach using intermediate variable - CheetahString clone is still needed due to borrow checker requirements - Leverages automatic coercion from &CheetahString to &str via AsRef/Deref - Makes code more idiomatic and readable - All tests passing
1 parent 90d2bba commit dd8c4e4

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

rocketmq-client/src/producer/default_mq_producer.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -692,7 +692,8 @@ impl DefaultMQProducer {
692692

693693
impl MQProducer for DefaultMQProducer {
694694
async fn start(&mut self) -> rocketmq_error::RocketMQResult<()> {
695-
let producer_group = self.with_namespace(self.producer_config.producer_group.clone().as_str());
695+
let producer_group_clone = self.producer_config.producer_group.clone();
696+
let producer_group = self.with_namespace(&producer_group_clone);
696697
self.set_producer_group(producer_group);
697698
let default_mqproducer_impl = self
698699
.default_mqproducer_impl

0 commit comments

Comments
 (0)