From ae7a9b0f4430a055ac419c6f1157cbd0e6eed45c Mon Sep 17 00:00:00 2001 From: Eli Schwartz Date: Sun, 9 Apr 2023 19:04:47 -0400 Subject: [PATCH] fix the repr for OrderedSet to actually execute correctly Old: ``` OrderedSet("'foo'", "'bar'", "'baz'") ``` New: ``` OrderedSet(['foo', 'bar', 'baz']) ``` The old one looked nasty *and* was totally non-functional. --- mesonbuild/utils/universal.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mesonbuild/utils/universal.py b/mesonbuild/utils/universal.py index 8216e780420c..26194628c46c 100644 --- a/mesonbuild/utils/universal.py +++ b/mesonbuild/utils/universal.py @@ -1886,8 +1886,8 @@ def __len__(self) -> int: def __repr__(self) -> str: # Don't print 'OrderedSet("")' for an empty set. if self.__container: - return 'OrderedSet("{}")'.format( - '", "'.join(repr(e) for e in self.__container.keys())) + return 'OrderedSet([{}])'.format( + ', '.join(repr(e) for e in self.__container.keys())) return 'OrderedSet()' def __reversed__(self) -> T.Iterator[_T]: