From 7c2489cc3dc798484e11a449fd0e305210de901b Mon Sep 17 00:00:00 2001
From: gcontini <1121667+gcontini@users.noreply.github.com>
Date: 周六, 14 12月 2019 20:12:41 +0800
Subject: [PATCH] fix codacy warnings

---
 cmake/Findlcc.cmake |  102 ++++++++++++++++++++++++++-------------------------
 1 files changed, 52 insertions(+), 50 deletions(-)

diff --git a/cmake/Findlcc.cmake b/cmake/Findlcc.cmake
index f38441f..bc994eb 100644
--- a/cmake/Findlcc.cmake
+++ b/cmake/Findlcc.cmake
@@ -1,45 +1,44 @@
-# Distributed under the OSI-approved BSD 3-Clause License.  See accompanying
-# file Copyright.txt or https://cmake.org/licensing for details.
+# Distributed under the OSI-approved BSD 3-Clause License.  
 
 #[=======================================================================[.rst:
-Findlcc
--------
-
-Find or build the lcc executable.
-
-Imported Targets
-^^^^^^^^^^^^^^^^
-
-This module provides the following imported targets, if found:
-
-``license_generator::lcc``
-  The lcc executable
-
-If lcc is not found this module will try to download it as a submodule
-Git must be installed.
-
-Input variables
-^^^^^^^^^^^^^^^^
-
-``LCC_LOCATION`` Hint for locating the lcc executable
-
-Result Variables
-^^^^^^^^^^^^^^^^
-
-This will define the following variables:
-
-``LCC_FOUND``
-  True if the system has the Foo library.
-``lcc_VERSION``
-
-Cache Variables
-^^^^^^^^^^^^^^^
-
-The following cache variables will also be set:
-
-``LCC_EXECUTABLE``
-  Path to the lcc executable.
-
+#Findlcc
+#-------
+#
+#Find or build the lcc executable.
+#
+#Imported Targets
+#^^^^^^^^^^^^^^^^
+#
+#This module provides the following imported targets, if found:
+#
+#``license_generator::lcc``
+#  The lcc executable
+#
+#If lcc is not found this module will try to download it as a submodule
+#Git must be installed.
+#
+#Input variables
+#^^^^^^^^^^^^^^^^
+#
+#``LCC_LOCATION`` Hint for locating the lcc executable
+#
+#Result Variables
+#^^^^^^^^^^^^^^^^
+#
+#This will define the following variables:
+#
+#``LCC_FOUND``
+#  True if the system has the Foo library.
+#``lcc_VERSION``
+#
+#Cache Variables
+#^^^^^^^^^^^^^^^
+#
+#The following cache variables will also be set:
+#
+#``LCC_EXECUTABLE``
+#  Path to the lcc executable.
+#
 #]=======================================================================]
 
 set(lcc_names lcc lcc.exe)
@@ -47,16 +46,18 @@
 find_package(PkgConfig)
 
 if(LCC_LOCATION)
-	# First search the PATH and specific locations.
-	find_program(LCC_EXECUTABLE
-	NAMES ${lcc_names} HINTS ${LCC_LOCATION} DOC "lcc command line client")
-	FIND_PACKAGE_HANDLE_STANDARD_ARGS(lcc FOUND_VAR LCC_FOUND
-                                       REQUIRED_VARS LCC_EXECUTABLE 
-                                       FAIL_MESSAGE "Error finding lcc executable. variable LCC_LOCATION non set correctly.")
-    add_executable(license_generator::lcc IMPORTED GLOBAL)                                            
-	set_property(TARGET license_generator::lcc PROPERTY IMPORTED_LOCATION ${LCC_EXECUTABLE})  
+	find_package(lcc HINTS ${LCC_LOCATION} CONFIG) #try to find it without looping on this module
+	if(NOT lcc_FOUND)
+		find_program(LCC_EXECUTABLE
+		NAMES ${lcc_names} HINTS ${LCC_LOCATION} DOC "lcc command line client")
+		FIND_PACKAGE_HANDLE_STANDARD_ARGS(lcc FOUND_VAR LCC_FOUND
+        	                               REQUIRED_VARS LCC_EXECUTABLE 
+            	                           FAIL_MESSAGE "Error finding lcc executable. variable LCC_LOCATION non set correctly.")
+    	add_executable(license_generator::lcc IMPORTED GLOBAL)                                            
+		set_property(TARGET license_generator::lcc PROPERTY IMPORTED_LOCATION ${LCC_EXECUTABLE})
+	ENDIF(NOT lcc_FOUND)
 ELSE(LCC_LOCATION)
-	find_package(lcc HINTS ${CMAKE_BINARY_DIR} 	NO_CMAKE_PATH) #try to find it without looping on this module
+	find_package(lcc HINTS ${CMAKE_BINARY_DIR} CONFIG) #try to find it without looping on this module
 
 	IF(NOT lcc_FOUND) 	
 		find_package(Git QUIET)
@@ -66,7 +67,7 @@
 		    if(GIT_SUBMODULE)
 		        message(STATUS "Submodule update")
 		        execute_process(COMMAND ${GIT_EXECUTABLE} submodule update --init --recursive
-		                        WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+		                        WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
 		                        RESULT_VARIABLE GIT_SUBMOD_RESULT)
 		        if(NOT GIT_SUBMOD_RESULT EQUAL "0")
 		            set(failure_messge  "git submodule update --init failed with ${GIT_SUBMOD_RESULT}, please checkout submodules")
@@ -77,6 +78,7 @@
 		    set(failure_messge  "All the options to find lcc executable failed. And i can't compile one from source GIT_SUBMODULE was turned off or failed. Please update submodules and try again.")
 		endif()
 		add_subdirectory("${PROJECT_SOURCE_DIR}/extern/license-generator")
+		set(lcc_FOUND TRUE)
 	ENDIF(NOT lcc_FOUND)
 ENDIF(LCC_LOCATION)
 

--
Gitblit v1.9.1