use string in AddCommandHandler

src/brpc/redis.cpp

@@ -436,9 +436,8 @@ std::ostream& operator<<(std::ostream& os, const RedisResponse& response) {
     return os;
 }
 
-bool RedisService::AddCommandHandler(const butil::StringPiece& name, 
-                                     RedisCommandHandler* handler) {
-    std::string lcname = StringToLowerASCII(name.as_string());
+bool RedisService::AddCommandHandler(const std::string& name, RedisCommandHandler* handler) {
+    std::string lcname = StringToLowerASCII(name);
     auto it = _command_map.find(lcname);
     if (it != _command_map.end()) {
         LOG(ERROR) << "redis command name=" << name << " exist";

src/brpc/redis.h

@@ -222,7 +222,7 @@ public:
     virtual ~RedisService() {}
     
     // Call this function to register `handler` that can handle command `name`.
-    bool AddCommandHandler(const butil::StringPiece& name, RedisCommandHandler* handler);
+    bool AddCommandHandler(const std::string& name, RedisCommandHandler* handler);
 
     // This function should not be touched by user and used by brpc deverloper only.
     RedisCommandHandler* FindCommandHandler(const butil::StringPiece& name) const;

src/brpc/redis_command.cpp

@@ -420,7 +420,7 @@ ParseError RedisCommandParser::Consume(butil::IOBuf& buf,
     char* d = (char*)arena->allocate((len/8 + 1) * 8);
     buf.cutn(d, len);
     d[len] = '\0';
-    _args[_index] = butil::StringPiece(d, len);
+    _args[_index].set(d, len);
     if (_index == 0) {
         // convert it to lowercase when it is command name
         for (int i = 0; i < len; ++i) {