The explicit call to the `std::string` constructor is for code expressiveness, it's not strictly necessary. A better idea might be to rename `buff` to something less generic, such as `deviceString`. Then you could also write
```cpp
return &deviceString[0];
```