Skip to content

Remove macro-redefined warning - #571

Open
pusewicz wants to merge 1 commit into
RandyGaul:masterfrom
pusewicz:define-ck-api
Open

Remove macro-redefined warning#571
pusewicz wants to merge 1 commit into
RandyGaul:masterfrom
pusewicz:define-ck-api

Conversation

@pusewicz

@pusewicz pusewicz commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

warning: 'CK_API' macro redefined [-Wmacro-redefined]

> warning: 'CK_API' macro redefined [-Wmacro-redefined]
Copilot AI lite review requested due to automatic review settings August 7, 2026 21:13
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Copilot AI 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.

Pull request overview

This PR aims to eliminate the Clang warning -Wmacro-redefined for CK_API by changing how CK_API/CM_API are defined in the framework’s core macro header (include/cute_defines.h).

Changes:

  • Guard CK_API and CM_API definitions with #ifndef to avoid redefining existing macros.

Comment thread include/cute_defines.h
Comment on lines +96 to +101
#ifndef CK_API
# define CK_API CF_API
#endif
#ifndef CM_API
# define CM_API CF_API
#endif
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