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

Jump to bottom

Distribute separate header files instead merged one [CORE4579] #4895

Closed
firebird-automations opened this issue Oct 15, 2014 · 9 comments
Closed

Distribute separate header files instead merged one [CORE4579] #4895

firebird-automations opened this issue Oct 15, 2014 · 9 comments
Labels
affect-version: 3.0 Beta 2 component: api / client library fix-version: 4.0 Beta 2 priority: minor qa: cannot be tested type: improvement

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @aafemt

Assigned to: @aafemt

When use merged ibase.h in a project that also uses ODBC interface, clash of macro definition happens. It can be avoided by precise using of separate header files.
@firebird-automations
Copy link
Collaborator Author

Commented by: @asfernandes

What clashes?

@firebird-automations
Copy link
Collaborator Author

Commented by: @aafemt

SQL_* constants.

@firebird-automations
Copy link
Collaborator Author

Commented by: @AlexPeshkoff

It would be useful to dup them with FB_SQL_* and enclose old defines into #⁠ifndef ....

@firebird-automations
Copy link
Collaborator Author

Modified by: @aafemt

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Beta 2 [ 10888 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: Cannot be tested

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

assignee: Dimitry Sibiryakov [ aafemt ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @dyemanov

I fail to see what commit or PR had fixed this ticket. Any hint?

@firebird-automations
Copy link
Collaborator Author

Commented by: @aafemt

PR #⁠205. Though I still think that dots in file path is ulgy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affect-version: 3.0 Beta 2 component: api / client library fix-version: 4.0 Beta 2 priority: minor qa: cannot be tested type: improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@firebird-automations