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 function declarations, for C99 compatibility #138

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

fweimer-rh
Copy link
Contributor

Add additional prototypes and #include directives for system headers, so that builds will not fail with compilers which do not support implicit function declarations by default.

Fixes some of the warnings in issue #29.

Related to:

Add additional prototypes and #include directives for system headers, so
that builds will not fail with compilers which do not support implicit
function declarations by default.
@lurcher
Copy link
Owner

lurcher commented Apr 13, 2023

None of the files modified are in the main code or are compiled by default so I have no worries with this change

@lurcher lurcher merged commit ac99c35 into lurcher:master Apr 13, 2023
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Aug 9, 2023
https://build.opensuse.org/request/show/1102884
by user jubalh + dimstar_suse
- Update to 2.3.12:
  * Makefile.am: Do not install config.h #108
  * DriverManager/drivermanager.h: fix build without threads #112
  * DriverManager/_info.c: Get locale encoding on Windows. #115
  * Fixed Connection String #127
  * Allow diagnostics to be retrieved on SQL_NO_DATA #137
  * Avoid implicit function declarations, for C99 compatibility #138
  * Remove self-reference #140
  * isql.1: Add information about handling passwords containing semicolons #126
- Remove unixODBC-gccwarnings.patch, unixODBC-2.3.6-declarations.patch:
  Changes have been done in lurcher/unixODBC#138
- Update unixODC-etc-location.patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants