strprintf - この実装は安全で移植性がありますか？

Question

私は、私たちのプロジェクトでsprintf/snprintfの代わりとして以下を提案することを考えています。

バッファサイズについて考える必要性を排除し、可能な限りオリジナルの便利さを維持することです。

std::string strprintf(const char *fromat, ...) 
{ 
    std::string s; 
    s.resize(128); // best guess 
    char *buff = const_cast<char *>(s.data()); 

    va_list arglist; 
    va_start(arglist, fromat); 
    auto len = vsnprintf(buff, 128, fromat, arglist); 
    va_end(arglist); 

    if (len > 127) 
    { 
     va_start(arglist, fromat); 
     s.resize(len + 1); // leave room for null terminator 
     buff = const_cast<char *>(s.data()); 
     len = vsnprintf(buff, len+1, fromat, arglist); 
     va_end(arglist); 
    } 
    s.resize(len); 
    return s; // move semantics FTW 
} 

このコードには固有の問題がありますか？

使用例：

auto s = strprintf("Hello %d world", 777); 

Answer 1

は、すでに多くの議論はここで、この上があることが判明：

std::string formatting like sprintf

主観的には、私のバージョンはまだほとんどのものよりもより簡潔に見えますが、それは機能的ですこのスレッドで提案されているソリューションの一部と同じです。