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

Fix 'pushKV ambiguous' error in compilation #15

Merged
merged 1 commit into from
Aug 30, 2021

Conversation

who-biz
Copy link

@who-biz who-biz commented Aug 30, 2021

  • Error occuring because there is no conversion function for size_t to C string in pushKV variatns
  • When conversion from size_t to uint64_t doesn't happen automatically on some systems, error is encountered
  • Affects at least M1 macOS

- Error occuring because there is no conversion function for size_t to C string in pushKV variatns
- When conversion from size_t to uint64_t doesn't happen automatically on some systems, error is encountered
- Affects at least M1 macOS
@who-biz
Copy link
Author

who-biz commented Aug 30, 2021

Please wait to merge this PR until we have confirmed there are no other areas where the same is attempted with a size_t conversion. Compilation is not showing errors on Ubuntu Server 18.04, so this seems to be related to platform-specific implementation of size type.

@satindergrewal satindergrewal merged commit 758e5d2 into satindergrewal:barry Aug 30, 2021
@satindergrewal
Copy link
Owner

oops 🙊
sorry, merged it too quick. should have read the second comment.

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.

2 participants