Skip to content

Fix: Define GPR_ASSERT macro for compatibility #660

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nikurt
Copy link

@nikurt nikurt commented May 16, 2025

Added a fallback definition for GPR_ASSERT using <assert.h> to handle cases where it is not predefined. This ensures smoother compilation and avoids potential issues with missing assertions.

Fixes #659

Copy link

ti-chi-bot bot commented May 16, 2025

Welcome @nikurt! It looks like this is your first PR to tikv/grpc-rs 🎉

@nikurt nikurt changed the title Define GPR_ASSERT macro to ensure compatibility. Fix: Define GPR_ASSERT macro for compatibility May 16, 2025
@nikurt nikurt closed this May 16, 2025
@nikurt nikurt reopened this May 16, 2025
Added a fallback definition for GPR_ASSERT using <assert.h> to handle cases where it is not predefined. This ensures smoother compilation and avoids potential issues with missing assertions.

Signed-off-by: Nikolay Kurtov <[email protected]>
Signed-off-by: Nikolay Kurtov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compilation error: GPR_ASSERT not found
1 participant