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
Use generic-number in riemann (4.0) #4052
Conversation
no news file needed. |
2176f47
to
6260aa7
Compare
Build SUCCESS |
6260aa7
to
4cf05d3
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.
I have checked the last commit, and it looks good to me.
I just want to mention that the old behavior was that g_scii_strtoll()
returned with 0
if it was unable to parse the string as number, meanwhile the new one only calls the setter if it was parsed successfully.
If #4022 gets merged, then it can get merged too (after a rebase, if needed).
4cf05d3
to
3267b67
Compare
Build FAILURE |
3267b67
to
547b1cd
Compare
@bazsi Can you rebase this branch for a last time, please? |
…ENT_FIELD_TIME Signed-off-by: Balazs Scheidler <bazsi77@gmail.com>
547b1cd
to
68ab80e
Compare
This PR changes the riemann driver to use GenericNumber and its associated number parsing functions, instead of using using strtoll() without error handling.
This is not really a functional change, rather it is a refactor, but has originally been done in the scope of the 4.0 typing work.