Skip to content

Commit ac2199b

Browse files
committed
BUG: Remove doxygen syntax markup for cmake lists
The DOXYGEN_INPUT variable had inconsistent mark up for elements. This PR removes the mark up for DOXYGEN_INPUT and DOXYGE_TEST_DIRS, to consistently allow the Doxygen CMake modules to handle the rendering of list of string.
1 parent c49d637 commit ac2199b

File tree

1 file changed

+6
-18
lines changed

1 file changed

+6
-18
lines changed

Utilities/Doxygen/CMakeLists.txt

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -24,30 +24,18 @@ set( DOX_MODULE_LIST ${ITK_MODULES_ENABLED} )
2424

2525
foreach(itk-module ${DOX_MODULE_LIST})
2626
if(${itk-module}_IS_TEST)
27-
list( APPEND DOXYGEN_TEST_DIRS "\"${${itk-module}_SOURCE_DIR}\" \\\n ")
27+
list( APPEND DOXYGEN_TEST_DIRS "${${itk-module}_SOURCE_DIR}")
2828
else()
2929
if( EXISTS ${${itk-module}_SOURCE_DIR}/include )
30-
list( APPEND DOXYGEN_INCLUDE_DIRS "\"${${itk-module}_SOURCE_DIR}/include\" \\\n")
30+
list( APPEND DOXYGEN_INCLUDE_DIRS "${${itk-module}_SOURCE_DIR}/include")
3131
endif()
3232
endif()
3333
endforeach()
3434

35-
list( APPEND DOXYGEN_TEST_DIRS "\"${ITK_SOURCE_DIR}/Examples\" \\\n")
36-
list( APPEND DOXYGEN_TEST_DIRS "\"${ITK_SOURCE_DIR}\" \\\n")
37-
list( APPEND DOXYGEN_TEST_DIRS " \"${ITK_SOURCE_DIR}/Modules/Remote\"\n")
35+
list( APPEND DOXYGEN_TEST_DIRS "${ITK_SOURCE_DIR}/Examples")
36+
list( APPEND DOXYGEN_TEST_DIRS "${ITK_SOURCE_DIR}")
37+
list( APPEND DOXYGEN_TEST_DIRS "${ITK_SOURCE_DIR}/Modules/Remote")
3838

39-
# By definition DOXYGEN_TEST_DIRS can not be empty. In the worst case if
40-
# BUILD_TESTING is turned OFF, there are at least 3 strings in
41-
# DOXYGEN_TEST_DIRS: ${ITK_SOURCE_DIR}/Examples, ${ITK_SOURCE_DIR},
42-
# and ${ITK_SOURCE_DIR}/Modules/Remote
43-
string( REPLACE ";" " \\\\n" DOXYGEN_TEST_DIRS ${DOXYGEN_TEST_DIRS} )
44-
45-
list( LENGTH DOXYGEN_INCLUDE_DIRS _doxygen_include_dirs_size )
46-
47-
if( ${_doxygen_include_dirs_size} GREATER 0 )
48-
# replace ";" in the string by space+backslash+newline
49-
string( REPLACE ";" " \\\\n" DOXYGEN_INCLUDE_DIRS ${DOXYGEN_INCLUDE_DIRS} )
50-
endif()
5139

5240
#
5341
# Configure the script and the doxyfile, then add target
@@ -145,7 +133,7 @@ if (ITK_BUILD_DOCUMENTATION)
145133
# -- Only build if 'Documentation' target is requested ALL
146134
WORKING_DIRECTORY ${ITK_BINARY_DIR}/Utilities/Doxygen
147135
# -- Some of the ITK_DOXYGEN_INPUT are directories, USE_STAMP_FILE requires only files be listed
148-
COMMENT "-- Wrapping ITK: Constructing documentation xml structure."
136+
COMMENT "-- Building ITK Doxygen."
149137
)
150138
unset(ITK_DOXYGEN_INPUT)
151139

0 commit comments

Comments
 (0)