Skip to content

test/params_test.c: diverse fixes#8469

Closed
levitte wants to merge 3 commits intoopenssl:masterfrom
levitte:fis-params_test-20190313
Closed

test/params_test.c: diverse fixes#8469
levitte wants to merge 3 commits intoopenssl:masterfrom
levitte:fis-params_test-20190313

Conversation

@levitte
Copy link
Member

@levitte levitte commented Mar 13, 2019

79c0b33 test/params_test.c: make construct_api_params() static
9eaf623 test/params_test.c: make more global variables static
f2003c8 test/params_test.c: use TEST_double_eq to check doubles

levitte added 2 commits March 13, 2019 13:56
With enough warning flags, compilers complain when a non-static
function hasn't been properly declared...
Again, compilers that don't like them being undeclared...
Copy link
Member

@romen romen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a nit!

TEST_ulong_eq was used previously because TEST_double_eq didn't exist
at the time.
@levitte levitte force-pushed the fis-params_test-20190313 branch from f2003c8 to 7145e85 Compare March 13, 2019 13:28
@levitte
Copy link
Member Author

levitte commented Mar 13, 2019

I'll just wait for the CIs to confirm as well...

@romen romen added the approval: done This pull request has the required number of approvals label Mar 13, 2019
@levitte
Copy link
Member Author

levitte commented Mar 13, 2019

Cis happy. Merged.

d362084 test/params_test.c: use TEST_double_eq to check doubles
932c3d0 test/params_test.c: make more global variables static
8472750 test/params_test.c: make construct_api_params() static

@levitte levitte closed this Mar 13, 2019
levitte added a commit that referenced this pull request Mar 13, 2019
With enough warning flags, compilers complain when a non-static
function hasn't been properly declared...

Reviewed-by: Nicola Tuveri <[email protected]>
(Merged from #8469)
levitte added a commit that referenced this pull request Mar 13, 2019
Again, compilers that don't like them being undeclared...

Reviewed-by: Nicola Tuveri <[email protected]>
(Merged from #8469)
levitte added a commit that referenced this pull request Mar 13, 2019
TEST_ulong_eq was used previously because TEST_double_eq didn't exist
at the time.

Reviewed-by: Nicola Tuveri <[email protected]>
(Merged from #8469)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants