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

avoid "implicit int" functions #2211

Closed
wants to merge 1 commit into from

Conversation

bbolli
Copy link

@bbolli bbolli commented Apr 12, 2021

The later definitions have a void return type, so use the same type in the "do-nothing" alternatives that are used e.g. when compiling for MinGW.

The later definitions have a void return type, so use the same type in
the "do-nothing" alternatives that are used e.g. when compiling for
MinGW.

Signed-off-by: Beat Bolli <dev@drbeat.li>
@codecov
Copy link

codecov bot commented Apr 12, 2021

Codecov Report

Merging #2211 (b9eb09c) into v2.x (42e368d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v2.x    #2211   +/-   ##
=======================================
  Coverage   88.78%   88.78%           
=======================================
  Files         139      139           
  Lines        5705     5705           
=======================================
  Hits         5065     5065           
  Misses        640      640           

@bbolli
Copy link
Author

bbolli commented Apr 13, 2021

Duplicate of #2213.

@bbolli bbolli closed this Apr 13, 2021
@bbolli bbolli deleted the v2.x-no-implicit-int branch April 13, 2021 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants