Kitware/CMake
1
0
Fork 0
mirror of https://github.com/Kitware/CMake.git synced 2025-10-19 02:17:27 +08:00
Code Issues Projects Releases Wiki Activity

list(INSERT): restore old behavior

Browse Source 
Fixes: #25191
This commit is contained in:
Marc Chevrier
2023-08-17 11:47:20 +02:00
parent a69c783749
commit 45ed175f08
5 changed files with 86 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Source/cmGeneratorExpressionNode.cxx
View File

@@ -1360,7 +1360,9 @@ static const struct ListNode : public cmGeneratorExpressionNode
try { try {
auto list = GetList(args.front()); auto list = GetList(args.front());
args.advance(2); args.advance(2);
list.insert_items(index, args.begin(), args.end()); list.insert_items(index, args.begin(), args.end(),
cmList::ExpandElements::No,
cmList::EmptyElements::Yes);
return list.to_string(); return list.to_string();
} catch (std::out_of_range& e) { } catch (std::out_of_range& e) {
reportError(ctx, cnt->GetOriginalExpression(), e.what()); reportError(ctx, cnt->GetOriginalExpression(), e.what());

15
Source/cmList.h
View File

@@ -740,9 +740,20 @@ public:
// Throw std::out_of_range if index is invalid // Throw std::out_of_range if index is invalid
template <typename InputIterator> template <typename InputIterator>
cmList& insert_items(index_type index, InputIterator first, cmList& insert_items(index_type index, InputIterator first,
InputIterator last) InputIterator last,
ExpandElements expandElements = ExpandElements::Yes,
EmptyElements emptyElements = EmptyElements::No)
{ {
this->insert(this->begin() + this->ComputeInsertIndex(index), first, last); this->insert(this->begin() + this->ComputeInsertIndex(index), first, last,
expandElements, emptyElements);
return *this;
}
template <typename InputIterator>
cmList& insert_items(index_type index, InputIterator first,
InputIterator last, EmptyElements emptyElements)
{
this->insert(this->begin() + this->ComputeInsertIndex(index), first, last,
ExpandElements::Yes, emptyElements);
return *this; return *this;
} }
cmList& insert_items(index_type index, cmList& insert_items(index_type index,

3
Source/cmListCommand.cxx
View File

@@ -375,7 +375,8 @@ bool HandleInsertCommand(std::vector<std::string> const& args,
} }
try { try {
list->insert_items(index, args.begin() + 3, args.end()); list->insert_items(index, args.begin() + 3, args.end(),
cmList::ExpandElements::No, cmList::EmptyElements::Yes);
status.GetMakefile().AddDefinition(listName, list->to_string()); status.GetMakefile().AddDefinition(listName, list->to_string());
return true; return true;
} catch (std::out_of_range& e) { } catch (std::out_of_range& e) {

21
Tests/CMakeTests/ListTest.cmake.in
View File

@@ -161,3 +161,24 @@ foreach (_pol ${thelist})
message(SEND_ERROR "returned element '${thevalue}', but expected '${_pol}'") message(SEND_ERROR "returned element '${thevalue}', but expected '${_pol}'")
endif() endif()
endforeach (_pol) endforeach (_pol)
block(SCOPE_FOR POLICIES)
cmake_policy(SET CMP0007 NEW)
set(result andy bill brad ken bob)
list(INSERT result 1 "")
TEST("INSERT result 1 \"\"" "andy;;bill;brad;ken;bob")
list(INSERT result 4 ";")
TEST("INSERT result 1 ;" "andy;;bill;brad;;;ken;bob")
list(INSERT result 0 "x")
TEST("INSERT result 1 x" "x;andy;;bill;brad;;;ken;bob")
endblock()
block(SCOPE_FOR POLICIES)
cmake_policy(SET CMP0007 OLD)
set(result andy bill brad ken bob)
list(INSERT result 1 "")
TEST("INSERT result 1 \"\"" "andy;;bill;brad;ken;bob")
list(INSERT result 4 ";")
TEST("INSERT result 1 ;" "andy;bill;brad;ken;;;bob")
list(INSERT result 0 "x")
TEST("INSERT result 1 x" "x;andy;bill;brad;ken;bob")
endblock()

47
Tests/RunCMake/GenEx-LIST/INSERT.cmake.in
View File

@@ -46,5 +46,52 @@ if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}") list (APPEND errors "returns bad value: ${output}")
endif() endif()
block(SCOPE_FOR POLICIES)
cmake_policy(SET CMP0007 NEW)
set(listvar "0;1;2;3;4")
list(INSERT listvar 1 "")
set (output "$<LIST:INSERT,0;1;2;3;4,1,>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
list(INSERT listvar 4 ";")
set (output "$<LIST:INSERT,0;;1;2;3;4,4,;>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
list(INSERT listvar 0 "x")
set (output "$<LIST:INSERT,0;;1;2;;;3;4,0,x>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
endblock()
block(SCOPE_FOR POLICIES)
set(CMAKE_WARN_DEPRECATED OFF CACHE BOOL "")
cmake_policy(SET CMP0007 OLD)
set(listvar "0;1;2;3;4")
list(INSERT listvar 1 "")
set (output "$<LIST:INSERT,0;1;2;3;4,1,>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
list(INSERT listvar 4 ";")
set (output "$<LIST:INSERT,0;1;2;3;4,4,;>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
list(INSERT listvar 0 "x")
set (output "$<LIST:INSERT,0;1;2;3;4,0,x>")
if (NOT output STREQUAL listvar)
list (APPEND errors "returns bad value: ${output}")
endif()
unset(CMAKE_WARN_DEPRECATED CACHE)
endblock()
check_errors("LIST:INSERT..." ${errors}) check_errors("LIST:INSERT..." ${errors})