dataroaring commented on PR #55950: URL: https://github.com/apache/doris/pull/55950#issuecomment-3284076971
**Missing Header Guards** **File**: `be/src/http/action/check_encryption_action.h` **Issue**: This header file lacks proper include guards, which can cause multiple inclusion problems during compilation. **Risk**: - Compilation errors in complex build scenarios - Potential symbol redefinition issues - Non-standard header practices **Fix**: Add header guards at the beginning of the file: ```cpp #pragma once // OR traditional guards: #ifndef BE_SRC_HTTP_ACTION_CHECK_ENCRYPTION_ACTION_H #define BE_SRC_HTTP_ACTION_CHECK_ENCRYPTION_ACTION_H // ... existing content ... #endif // BE_SRC_HTTP_ACTION_CHECK_ENCRYPTION_ACTION_H ``` Recommend using `#pragma once` as it's simpler and widely supported by modern compilers. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
