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.

Sign up for GitHub

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

Subject to multiple definition errors. #1

Closed
midn1 opened this issue May 21, 2018 · 1 comment
Closed

Subject to multiple definition errors. #1

midn1 opened this issue May 21, 2018 · 1 comment
Labels
bug Something isn't working

Comments

@midn1
Copy link

midn1 commented May 21, 2018

As this library is header-only, shouldn't all the functions be inline to prevent multiple definition errors?
@metayeti
Copy link
Owner

You're correct - from what I understand, only static functions need to be inlined though. Will be fixed in the next version. Thanks for reporting this.

@metayeti metayeti added the bug Something isn't working label May 21, 2018
metayeti pushed a commit that referenced this issue Jan 11, 2024
@MabelYT
Merge pull request #1 from MabelYT/mabelisyt-0915
97516fe 
Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@metayeti @midn1