Skip to content

Commit 162f996

Browse files
committed
lint
1 parent 0a986d4 commit 162f996

File tree

3 files changed

+20
-7
lines changed

3 files changed

+20
-7
lines changed

cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2644,7 +2644,9 @@ if(ARROW_WITH_SIMDJSON)
26442644
set_property(TARGET simdjson::simdjson
26452645
APPEND
26462646
PROPERTY INTERFACE_COMPILE_DEFINITIONS SIMDJSON_HEADER_ONLY)
2647-
set(ARROW_SIMDJSON_FORCE_HEADER_ONLY TRUE CACHE INTERNAL "Force simdjson header-only mode")
2647+
set(ARROW_SIMDJSON_FORCE_HEADER_ONLY
2648+
TRUE
2649+
CACHE INTERNAL "Force simdjson header-only mode")
26482650
endif()
26492651
endif()
26502652

cpp/src/arrow/CMakeLists.txt

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -685,17 +685,20 @@ if(TARGET simdjson::simdjson)
685685
message(STATUS "simdjson target found:")
686686
message(STATUS " Type: ${SIMDJSON_TARGET_TYPE}")
687687
message(STATUS " Interface definitions: ${SIMDJSON_INTERFACE_DEFINITIONS}")
688-
message(STATUS " ARROW_SIMDJSON_FORCE_HEADER_ONLY: ${ARROW_SIMDJSON_FORCE_HEADER_ONLY}")
688+
message(STATUS " ARROW_SIMDJSON_FORCE_HEADER_ONLY: ${ARROW_SIMDJSON_FORCE_HEADER_ONLY}"
689+
)
689690

690691
# Determine if header-only mode is needed
691-
if(SIMDJSON_INTERFACE_DEFINITIONS AND "SIMDJSON_HEADER_ONLY" IN_LIST SIMDJSON_INTERFACE_DEFINITIONS)
692+
if(SIMDJSON_INTERFACE_DEFINITIONS AND "SIMDJSON_HEADER_ONLY" IN_LIST
693+
SIMDJSON_INTERFACE_DEFINITIONS)
692694
message(STATUS "simdjson: header-only mode detected via interface definitions")
693695
set(ARROW_SIMDJSON_HEADER_ONLY TRUE)
694696
elseif(SIMDJSON_TARGET_TYPE STREQUAL "INTERFACE_LIBRARY")
695697
message(STATUS "simdjson: header-only mode detected via INTERFACE_LIBRARY type")
696698
set(ARROW_SIMDJSON_HEADER_ONLY TRUE)
697699
elseif(ARROW_SIMDJSON_FORCE_HEADER_ONLY)
698-
message(STATUS "simdjson: header-only mode forced via ARROW_SIMDJSON_FORCE_HEADER_ONLY")
700+
message(STATUS "simdjson: header-only mode forced via ARROW_SIMDJSON_FORCE_HEADER_ONLY"
701+
)
699702
set(ARROW_SIMDJSON_HEADER_ONLY TRUE)
700703
else()
701704
# Check if the simdjson target has actual library files to link against
@@ -715,7 +718,12 @@ if(TARGET simdjson::simdjson)
715718
endforeach()
716719
endif()
717720
if(NOT SIMDJSON_HAS_LIB)
718-
foreach(CONFIG "" "_DEBUG" "_RELEASE" "_RELWITHDEBINFO" "_MINSIZEREL")
721+
foreach(CONFIG
722+
""
723+
"_DEBUG"
724+
"_RELEASE"
725+
"_RELWITHDEBINFO"
726+
"_MINSIZEREL")
719727
get_target_property(_loc simdjson::simdjson "IMPORTED_LOCATION${CONFIG}")
720728
get_target_property(_imp simdjson::simdjson "IMPORTED_IMPLIB${CONFIG}")
721729
if(_loc OR _imp)

cpp/src/arrow/integration/CMakeLists.txt

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,11 @@ endif()
3030

3131
if(ARROW_BUILD_TESTS)
3232
add_arrow_test(json_integration_test
33-
EXTRA_LINK_LIBS simdjson::simdjson ${GFLAGS_LIBRARIES}
34-
DEFINITIONS ${ARROW_INTEGRATION_TEST_DEFINITIONS})
33+
EXTRA_LINK_LIBS
34+
simdjson::simdjson
35+
${GFLAGS_LIBRARIES}
36+
DEFINITIONS
37+
${ARROW_INTEGRATION_TEST_DEFINITIONS})
3538
add_dependencies(arrow-integration arrow-json-integration-test)
3639
elseif(ARROW_BUILD_INTEGRATION)
3740
add_executable(arrow-json-integration-test json_integration_test.cc)

0 commit comments

Comments
 (0)