New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add StringToDoubleConverter::StringTo<T>
member function templates
#158
Add StringToDoubleConverter::StringTo<T>
member function templates
#158
Conversation
41d3f37
to
71e59bf
Compare
Allowed users to write generic code more easily, like this (when `T` is either `float` or `double`): converter.StringTo<T>(buffer, length, &processed); Included a unit test, `TEST(StringToTemplate)`. Fixes issue google#157 With help from Florian Loitsch (floitsch)
71e59bf
to
6a55772
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
thanks!
@floitsch Thanks a lot for merging my very first double-conversion pull request, Florian! You're very helpful! (It's actually my second merged PR at github.com/google, as my PR google/googletest#3184 was merged just last month 😃!) I'm now considering a similar PR for |
Allowed users to write generic code more easily, in cases where the input value is either a double, or a single-precision float. Included a unit test, `TEST(DoubleToShortestString)`. Complementary to pull request google#158 commit eee1a45 "Add `StringToDoubleConverter::StringTo<T>` member function templates"
Allowed users to write generic code more easily, in cases where the input value is either a double, or a single-precision float. Included a unit test, `TEST(DoubleToShortestString)`. Complementary to pull request google#158 commit eee1a45 "Add `StringToDoubleConverter::StringTo<T>` member function templates"
Allowed users to write generic code more easily, in cases where the input value is either a double, or a single-precision float. Included a unit test, `TEST(DoubleToShortestString)`. Complementary to pull request google#158 commit eee1a45 "Add `StringToDoubleConverter::StringTo<T>` member function templates"
Allowed users to write generic code more easily, like this (when
T
iseither
float
ordouble
):Included a unit test,
TEST(StringToTemplate)
.Fixes issue #157
With help from Florian Loitsch (@floitsch)