From aa0aaa268126848ac7402c9ddbf405f513312adc Mon Sep 17 00:00:00 2001 From: Karol Grydziuszko Date: Wed, 22 Dec 2021 15:37:03 +0100 Subject: [PATCH 1/4] Removed dependency from libhilog and added dependency from libhilog_base Signed-off-by: Karol Grydziuszko Change-Id: I2ece90a248bf568e1ccfb1c4e234704a8258d46c --- base/BUILD.gn | 4 ++-- base/src/utils_log.h | 39 ++++++++++++++++++++++++++------------- 2 files changed, 28 insertions(+), 15 deletions(-) diff --git a/base/BUILD.gn b/base/BUILD.gn index cdf421d..883b1ec 100644 --- a/base/BUILD.gn +++ b/base/BUILD.gn @@ -110,7 +110,7 @@ ohos_static_library("utilsbase") { ] public_configs = [ ":static_utils_config" ] defines = [ "CONFIG_HILOG" ] - external_deps = [ "hilog_native:libhilog" ] + external_deps = [ "hilog_native:libhilog_base" ] } ohos_shared_library("utils") { @@ -123,7 +123,7 @@ ohos_shared_library("utils") { public_configs = [ ":static_utils_config" ] subsystem_name = "utils" defines = [ "CONFIG_HILOG" ] - external_deps = [ "hilog_native:libhilog" ] + external_deps = [ "hilog_native:libhilog_base" ] part_name = "utils_base" install_images = [ "system", diff --git a/base/src/utils_log.h b/base/src/utils_log.h index 7c2386c..082d97c 100644 --- a/base/src/utils_log.h +++ b/base/src/utils_log.h @@ -16,13 +16,17 @@ #define UTILS_BASE_LOG_H #ifdef CONFIG_HILOG -#include "hilog/log.h" -static constexpr OHOS::HiviewDFX::HiLogLabel label = { LOG_CORE, 0xD003D00, "utils_base" }; -#define UTILS_LOGF(...) (void)OHOS::HiviewDFX::HiLog::Fatal(label, __VA_ARGS__) -#define UTILS_LOGE(...) (void)OHOS::HiviewDFX::HiLog::Error(label, __VA_ARGS__) -#define UTILS_LOGW(...) (void)OHOS::HiviewDFX::HiLog::Warn(label, __VA_ARGS__) -#define UTILS_LOGI(...) (void)OHOS::HiviewDFX::HiLog::Info(label, __VA_ARGS__) -#define UTILS_LOGD(...) (void)OHOS::HiviewDFX::HiLog::Debug(label, __VA_ARGS__) +#include "hilog_base/log_base.h" +namespace UTILS_LOG { +constexpr LogType TYPE = LOG_CORE; +constexpr unsigned int DOMAIN = 0xD003D00; +constexpr const char *TAG = "utils_base"; +} // namespace UTILS_LOG +#define UTILS_LOGF(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_FATAL, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) +#define UTILS_LOGE(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_ERROR, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) +#define UTILS_LOGW(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_WARN, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) +#define UTILS_LOGI(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_INFO, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) +#define UTILS_LOGD(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_DEBUG, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) #else #define UTILS_LOGF(...) #define UTILS_LOGE(...) @@ -32,12 +36,21 @@ static constexpr OHOS::HiviewDFX::HiLogLabel label = { LOG_CORE, 0xD003D00, "uti #endif // CONFIG_HILOG #if (defined CONFIG_HILOG) && (defined CONFIG_PARCEL_DEBUG) -static constexpr OHOS::HiviewDFX::HiLogLabel parcelLabel = { LOG_CORE, 0xD003D01, "parcel" }; -#define PARCEL_LOGF(...) (void)OHOS::HiviewDFX::HiLog::Fatal(parcelLabel, __VA_ARGS__) -#define PARCEL_LOGE(...) (void)OHOS::HiviewDFX::HiLog::Error(parcelLabel, __VA_ARGS__) -#define PARCEL_LOGW(...) (void)OHOS::HiviewDFX::HiLog::Warn(parcelLabel, __VA_ARGS__) -#define PARCEL_LOGI(...) (void)OHOS::HiviewDFX::HiLog::Info(parcelLabel, __VA_ARGS__) -#define PARCEL_LOGD(...) (void)OHOS::HiviewDFX::HiLog::Debug(parcelLabel, __VA_ARGS__) +namespace PARCEL_LOG { +constexpr LogType TYPE = LOG_CORE; +constexpr unsigned int DOMAIN = 0xD003D01; +constexpr const char *TAG = "parcel"; +} // namespace PARCEL_LOG +#define PARCEL_LOGF(...) \ + (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_FATAL, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) +#define PARCEL_LOGE(...) \ + (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_ERROR, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) +#define PARCEL_LOGW(...) \ + (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_WARN, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) +#define PARCEL_LOGI(...) \ + (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_INFO, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) +#define PARCEL_LOGD(...) \ + (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_DEBUG, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) #else #define PARCEL_LOGF(...) #define PARCEL_LOGE(...) -- Gitee From fd4ebff74506323e535f73c8a1649190bab61fba Mon Sep 17 00:00:00 2001 From: Karol Grydziuszko Date: Thu, 23 Dec 2021 19:02:25 +0100 Subject: [PATCH 2/4] Changed namespace to camel case Signed-off-by: Karol Grydziuszko Change-Id: I9fecb1db85337156b206ab66e34c1a6e427d43c8 --- base/src/utils_log.h | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/base/src/utils_log.h b/base/src/utils_log.h index 082d97c..6bf572a 100644 --- a/base/src/utils_log.h +++ b/base/src/utils_log.h @@ -17,16 +17,16 @@ #ifdef CONFIG_HILOG #include "hilog_base/log_base.h" -namespace UTILS_LOG { +namespace UtilsLog { constexpr LogType TYPE = LOG_CORE; constexpr unsigned int DOMAIN = 0xD003D00; constexpr const char *TAG = "utils_base"; -} // namespace UTILS_LOG -#define UTILS_LOGF(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_FATAL, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) -#define UTILS_LOGE(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_ERROR, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) -#define UTILS_LOGW(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_WARN, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) -#define UTILS_LOGI(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_INFO, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) -#define UTILS_LOGD(...) (void)HiLogBasePrint(UTILS_LOG::TYPE, LOG_DEBUG, UTILS_LOG::DOMAIN, UTILS_LOG::TAG, __VA_ARGS__) +} // namespace UtilsLog +#define UTILS_LOGF(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_FATAL, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) +#define UTILS_LOGE(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_ERROR, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) +#define UTILS_LOGW(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_WARN, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) +#define UTILS_LOGI(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_INFO, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) +#define UTILS_LOGD(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_DEBUG, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) #else #define UTILS_LOGF(...) #define UTILS_LOGE(...) @@ -36,21 +36,21 @@ constexpr const char *TAG = "utils_base"; #endif // CONFIG_HILOG #if (defined CONFIG_HILOG) && (defined CONFIG_PARCEL_DEBUG) -namespace PARCEL_LOG { +namespace ParcelLog { constexpr LogType TYPE = LOG_CORE; constexpr unsigned int DOMAIN = 0xD003D01; constexpr const char *TAG = "parcel"; -} // namespace PARCEL_LOG +} // namespace ParcelLog #define PARCEL_LOGF(...) \ - (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_FATAL, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) + (void)HiLogBasePrint(ParcelLog::TYPE, LOG_FATAL, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) #define PARCEL_LOGE(...) \ - (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_ERROR, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) + (void)HiLogBasePrint(ParcelLog::TYPE, LOG_ERROR, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) #define PARCEL_LOGW(...) \ - (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_WARN, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) + (void)HiLogBasePrint(ParcelLog::TYPE, LOG_WARN, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) #define PARCEL_LOGI(...) \ - (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_INFO, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) + (void)HiLogBasePrint(ParcelLog::TYPE, LOG_INFO, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) #define PARCEL_LOGD(...) \ - (void)HiLogBasePrint(PARCEL_LOG::TYPE, LOG_DEBUG, PARCEL_LOG::DOMAIN, PARCEL_LOG::TAG, __VA_ARGS__) + (void)HiLogBasePrint(ParcelLog::TYPE, LOG_DEBUG, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) #else #define PARCEL_LOGF(...) #define PARCEL_LOGE(...) -- Gitee From 8b36b3acb6955d666f36d0c85a8a217d896311d0 Mon Sep 17 00:00:00 2001 From: Karol Grydziuszko Date: Mon, 27 Dec 2021 08:46:50 +0100 Subject: [PATCH 3/4] Removed namespaces. Signed-off-by: Karol Grydziuszko Change-Id: I83c8d71fafea42afefe3b73e546fba11535dbcfb --- base/src/utils_log.h | 36 ++++++++++++++++-------------------- 1 file changed, 16 insertions(+), 20 deletions(-) diff --git a/base/src/utils_log.h b/base/src/utils_log.h index 6bf572a..9fda24f 100644 --- a/base/src/utils_log.h +++ b/base/src/utils_log.h @@ -17,16 +17,14 @@ #ifdef CONFIG_HILOG #include "hilog_base/log_base.h" -namespace UtilsLog { -constexpr LogType TYPE = LOG_CORE; -constexpr unsigned int DOMAIN = 0xD003D00; -constexpr const char *TAG = "utils_base"; -} // namespace UtilsLog -#define UTILS_LOGF(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_FATAL, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) -#define UTILS_LOGE(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_ERROR, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) -#define UTILS_LOGW(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_WARN, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) -#define UTILS_LOGI(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_INFO, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) -#define UTILS_LOGD(...) (void)HiLogBasePrint(UtilsLog::TYPE, LOG_DEBUG, UtilsLog::DOMAIN, UtilsLog::TAG, __VA_ARGS__) +constexpr LogType UTILS_LOG_TYPE = LOG_CORE; +constexpr unsigned int UTILS_LOG_DOMAIN = 0xD003D00; +constexpr const char *UTILS_LOG_TAG = "utils_base"; +#define UTILS_LOGF(...) (void)HiLogBasePrint(UTILS_LOG_TYPE, LOG_FATAL, UTILS_LOG_DOMAIN, UTILS_LOG_TAG, __VA_ARGS__) +#define UTILS_LOGE(...) (void)HiLogBasePrint(UTILS_LOG_TYPE, LOG_ERROR, UTILS_LOG_DOMAIN, UTILS_LOG_TAG, __VA_ARGS__) +#define UTILS_LOGW(...) (void)HiLogBasePrint(UTILS_LOG_TYPE, LOG_WARN, UTILS_LOG_DOMAIN, UTILS_LOG_TAG, __VA_ARGS__) +#define UTILS_LOGI(...) (void)HiLogBasePrint(UTILS_LOG_TYPE, LOG_INFO, UTILS_LOG_DOMAIN, UTILS_LOG_TAG, __VA_ARGS__) +#define UTILS_LOGD(...) (void)HiLogBasePrint(UTILS_LOG_TYPE, LOG_DEBUG, UTILS_LOG_DOMAIN, UTILS_LOG_TAG, __VA_ARGS__) #else #define UTILS_LOGF(...) #define UTILS_LOGE(...) @@ -36,21 +34,19 @@ constexpr const char *TAG = "utils_base"; #endif // CONFIG_HILOG #if (defined CONFIG_HILOG) && (defined CONFIG_PARCEL_DEBUG) -namespace ParcelLog { -constexpr LogType TYPE = LOG_CORE; -constexpr unsigned int DOMAIN = 0xD003D01; -constexpr const char *TAG = "parcel"; -} // namespace ParcelLog +constexpr LogType PARCEL_LOG_TYPE = LOG_CORE; +constexpr unsigned int PARCEL_LOG_DOMAIN = 0xD003D01; +constexpr const char *PARCEL_LOG_TAG = "parcel"; #define PARCEL_LOGF(...) \ - (void)HiLogBasePrint(ParcelLog::TYPE, LOG_FATAL, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) + (void)HiLogBasePrint(PARCEL_LOG_TYPE, LOG_FATAL, PARCEL_LOG_DOMAIN, PARCEL_LOG_TAG, __VA_ARGS__) #define PARCEL_LOGE(...) \ - (void)HiLogBasePrint(ParcelLog::TYPE, LOG_ERROR, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) + (void)HiLogBasePrint(PARCEL_LOG_TYPE, LOG_ERROR, PARCEL_LOG_DOMAIN, PARCEL_LOG_TAG, __VA_ARGS__) #define PARCEL_LOGW(...) \ - (void)HiLogBasePrint(ParcelLog::TYPE, LOG_WARN, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) + (void)HiLogBasePrint(PARCEL_LOG_TYPE, LOG_WARN, PARCEL_LOG_DOMAIN, PARCEL_LOG_TAG, __VA_ARGS__) #define PARCEL_LOGI(...) \ - (void)HiLogBasePrint(ParcelLog::TYPE, LOG_INFO, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) + (void)HiLogBasePrint(PARCEL_LOG_TYPE, LOG_INFO, PARCEL_LOG_DOMAIN, PARCEL_LOG_TAG, __VA_ARGS__) #define PARCEL_LOGD(...) \ - (void)HiLogBasePrint(ParcelLog::TYPE, LOG_DEBUG, ParcelLog::DOMAIN, ParcelLog::TAG, __VA_ARGS__) + (void)HiLogBasePrint(PARCEL_LOG_TYPE, LOG_DEBUG, PARCEL_LOG_DOMAIN, PARCEL_LOG_TAG, __VA_ARGS__) #else #define PARCEL_LOGF(...) #define PARCEL_LOGE(...) -- Gitee From ca2357187982beb9806f1a900c778f5b276b8487 Mon Sep 17 00:00:00 2001 From: Karol Grydziuszko Date: Mon, 27 Dec 2021 14:45:55 +0100 Subject: [PATCH 4/4] Fixed dependency relation to add libsec_shared to img files. Signed-off-by: Karol Grydziuszko Change-Id: I3deb6c9a432e029b96d9761c1dbc255c010ac8f3 --- ohos.build | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ohos.build b/ohos.build index 181551b..b671cb1 100644 --- a/ohos.build +++ b/ohos.build @@ -5,7 +5,8 @@ "module_list": [ "//utils/native/base:utils", "//utils/native/ndk_libraries_config:ndk_libs_config", - "//utils/native/base:utilsecurec_shared" + "//utils/native/base:utilsecurec_shared", + "//third_party/bounds_checking_function:libsec_shared" ], "inner_kits": [ { -- Gitee