Skip to content
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

IsDouble + string2double #1306

Closed
kzangeli opened this issue Sep 30, 2015 · 1 comment
Closed

IsDouble + string2double #1306

kzangeli opened this issue Sep 30, 2015 · 1 comment
Assignees
Milestone

Comments

@kzangeli
Copy link
Member

Now that we have a possible solution to the problem about assuring a string is a 'double', let's add a few functions to the 'common' lib:

bool isDouble(char*);
bool string2double(char* s, double* dP);  // return FALSE if 's' does NOT contain a double

Also, all uses of atof, strtod and similar functions must be looked up and made use these new functions instead of whatever they are using right now.

Already known cases:
Case 1: PUT .../attribute/value with Content-Type text (See PR #1299)
Case 2: Fermin had a similar problem not long ago - there we probably can use these new functions as well
Case X: any other uses of atof, strtod etc ...

Effort: 1 man day

@fgalan
Copy link
Member

fgalan commented Feb 4, 2016

Fixed in PR #1762

@fgalan fgalan closed this as completed Feb 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants