From c08576c0ced2c0bcff76d2f064e0cabf4c80916a Mon Sep 17 00:00:00 2001
From: ring_king <1300547600@qq.com>
Date: Wed, 21 Feb 2024 09:01:32 +0000
Subject: [PATCH] fix code rules

Signed-off-by: ring_king <1300547600@qq.com>
---
 src/gn/innerapis_publicinfo_generator.cc | 1 -
 src/gn/ohos_components.cc                | 4 ++--
 src/gn/ohos_components_checker.cc        | 1 -
 src/gn/ohos_components_mapping.cc        | 3 +--
 4 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/gn/innerapis_publicinfo_generator.cc b/src/gn/innerapis_publicinfo_generator.cc
index 7405845..682cf54 100644
--- a/src/gn/innerapis_publicinfo_generator.cc
+++ b/src/gn/innerapis_publicinfo_generator.cc
@@ -5,7 +5,6 @@
 #include "gn/innerapis_publicinfo_generator.h"
 
 #include <fstream>
-#include <filesystem>
 #include <iostream>
 #include <sys/stat.h>
 
diff --git a/src/gn/ohos_components.cc b/src/gn/ohos_components.cc
index 706439c..a074037 100644
--- a/src/gn/ohos_components.cc
+++ b/src/gn/ohos_components.cc
@@ -48,7 +48,7 @@ void OhosComponent::addInnerApi(const std::string &name, const std::string &labe
 {
     std::string la = label;
     size_t pos = label.find(":");
-    if (pos > 0) {
+    if (pos != std::string::npos) {
         if ((label[pos - 1]) == '/') { // some are like this : "//components/foo/libfoo/:libfoo"
             unsigned long indexToRemove = pos - 1;
             if (indexToRemove >= 0 && indexToRemove <= la.length()) {
@@ -341,7 +341,7 @@ bool OhosComponentsImpl::GetExternalDepsLabel(const Value &external_dep, std::st
 {
     std::string str_val = external_dep.string_value();
     size_t sep = str_val.find(":");
-    if (sep <= 0) {
+    if (sep == std::string::npos) {
         *err = Err(external_dep, "OHOS component external_deps format error: (" + external_dep.string_value() +
             "),"
             "it should be a string like \"component_name:innerapi_name\".");
diff --git a/src/gn/ohos_components_checker.cc b/src/gn/ohos_components_checker.cc
index a93eb2b..fc81c7b 100644
--- a/src/gn/ohos_components_checker.cc
+++ b/src/gn/ohos_components_checker.cc
@@ -4,7 +4,6 @@
 
 #include "gn/ohos_components_checker.h"
 
-#include <filesystem>
 #include <fstream>
 #include <functional>
 #include <iostream>
diff --git a/src/gn/ohos_components_mapping.cc b/src/gn/ohos_components_mapping.cc
index bcb4c2c..b1064a4 100644
--- a/src/gn/ohos_components_mapping.cc
+++ b/src/gn/ohos_components_mapping.cc
@@ -4,7 +4,6 @@
 
 #include "gn/ohos_components_mapping.h"
 
-#include <filesystem>
 #include <fstream>
 #include <functional>
 #include <iostream>
@@ -107,7 +106,7 @@ const std::string OhosComponentMapping::MappingTargetAbsoluteDpes(const BuildSet
     }
 
     size_t pos = deps.find(":");
-    if (pos <= 0) {
+    if (pos == std::string::npos) {
         return "";
     }
     return deps_component->getInnerApi(deps.substr(pos + 1));
-- 
Gitee