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(C): Update common.h #1278

Merged
merged 1 commit into from
Apr 15, 2024
Merged

fix(C): Update common.h #1278

merged 1 commit into from
Apr 15, 2024

Conversation

nuquist19
Copy link
Contributor

一个非常小的建议,请问这里为了防止重复包含头文件是不是应该改成COMMON_H,因为头文件名字叫做common.h

If this pull request (PR) pertains to Chinese-to-English translation, please confirm that you have read the contribution guidelines and complete the checklist below:

  • This PR represents the translation of a single, complete document, or contains only bug fixes.
  • The translation accurately conveys the original meaning and intent of the Chinese version. If deviations exist, I have provided explanatory comments to clarify the reasons.

If this pull request (PR) is associated with coding or code transpilation, please attach the relevant console outputs to the PR and complete the following checklist:

  • I have thoroughly reviewed the code, focusing on its formatting, comments, indentation, and file headers.
  • I have confirmed that the code execution outputs are consistent with those produced by the reference code (Python or Java).
  • The code is designed to be compatible on standard operating systems, including Windows, macOS, and Ubuntu.

请问这里为了防止重复包含头文件是不是应该改成COMMON_H,因为头文件名字叫做common.h
Copy link
Owner

@krahets krahets left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@krahets krahets changed the title Update common.h fix(C): Update common.h Apr 15, 2024
@krahets krahets added bug Bug fixes code Code-related labels Apr 15, 2024
@krahets krahets merged commit 2b11639 into krahets:main Apr 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fixes code Code-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants