Skip to content

hexify helper method name in test changed to char2hex - #6629

Merged
ev-mp merged 1 commit into
realsenseai:developmentfrom
remibettan:fix-linkage-hexify-method
Jun 18, 2020
Merged

hexify helper method name in test changed to char2hex#6629
ev-mp merged 1 commit into
realsenseai:developmentfrom
remibettan:fix-linkage-hexify-method

Conversation

@remibettan

Copy link
Copy Markdown
Contributor

Same name of helper function in global namespace lead to failure in linkage.

@ev-mp ev-mp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why C&P instead or reuse the original function? Disregard this

@remibettan
remibettan force-pushed the fix-linkage-hexify-method branch from 96f1838 to 721a2a1 Compare June 18, 2020 08:08

@ev-mp ev-mp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For now types.h. will remain private.

@ev-mp
ev-mp merged commit d6b9c65 into realsenseai:development Jun 18, 2020
@remibettan
remibettan deleted the fix-linkage-hexify-method branch July 29, 2025 14:08
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