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

INTERNAL: Fix a hidden keyword compile error #340

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

ing-eoking
Copy link
Collaborator

⌨️ What I did

  • MacOS(clang 16.0.6)에서 발생하는 컴파일 에러를 수정합니다.
  • 발생하는 컴파일 에러는 아래와 같습니다.
libhashkit/jenkins.cc:30:9: error: keyword is hidden by macro definition [-Werror,-Wkeyword-macro]
#define final(a,b,c) \
  • C++에서 final 키워드는 Java와 유사하게 상속을 불가하게 하거나 메서드의 오버라이딩을 막는 용도로 사용됩니다.
  • 이를 해결하기 위해 final 매크로를 JENKINS_FINAL로 변경했습니다.
    • 비슷하게 mix 매크로도 JENKINS_MIX로 변경하였습니다.
    • rot 매크로는 일반적으로 사용되는 비트 회전 방식과 유사하여, 이름을 변경하지 않았습니다.

@jhpark816 jhpark816 merged commit 6edb5f7 into naver:develop Jan 21, 2025
1 check passed
@ing-eoking ing-eoking deleted the jenkins branch January 21, 2025 02:00
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