You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently there is an inconsistency in function naming. Some functions are named according to Google C++ Style Guide:
Format
Print
Clear
ReportSystemError
...
Other follow standard library conventions:
c_str
str
clear
resize
reserve
printf
sprintf
...
The names should be unified before the version 1.0 of the API is released. This is pretty arbitrary, but it's probably better to follow standard library conventions for public API for consistency with printf and because its done already for most functions.
To minimize the number of API changes, it might be useful to consider simplification of a common use case of formatting into an std::string here too. Currently it requires two function calls:
str(Format("{}", 42));
Ideally it should require a single function call. Possible ways to do this:
Make format return std::string.
Add another method (strformat) returning std::string.
Activity
vitaut commentedon Jun 29, 2014
To minimize the number of API changes, it might be useful to consider simplification of a common use case of formatting into an
std::stringhere too. Currently it requires two function calls:Ideally it should require a single function call. Possible ways to do this:
formatreturnstd::string.strformat) returningstd::string.vitaut commentedon Jul 2, 2014
Fixed in version 0.9.0.
CheckPtr -> make_ptr (#50)
FormatFloat -> format_float (#50)
ReportUnknownType -> report_unknown_type (#50)
FormatDecimal -> format_decimal (#50)
More fixes for #50.
More fixes for #50.
More fixes for #50.
Fix Fix #50 in tests too.