Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
Subject: Re: svn commit: r1743195 -
 /subversion/trunk/subversion/tests/cmdline/svntest/main.py
To: dev@subversion.apache.org
References: <20160510142957.39F873A0216@svn01-us-west.apache.org>
 <5731F167.5000903@apache.org>
From: Stefan Fuhrmann <stefanfuhrmann@alice-dsl.de>
Message-ID: <57323275.3020309@alice-dsl.de>
Date: Tue, 10 May 2016 21:11:49 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.7.2
MIME-Version: 1.0
In-Reply-To: <5731F167.5000903@apache.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
archived-at: Tue, 10 May 2016 19:12:04 -0000

On 10.05.2016 16:34, Branko Čibej wrote:
> On 10.05.2016 16:29, stefan2@apache.org wrote:
>> Author: stefan2
>> Date: Tue May 10 14:29:56 2016
>> New Revision: 1743195
>>
>> URL: http://svn.apache.org/viewvc?rev=1743195&view=rev
>> Log:
>> Follow-up to r1743183:  Unbreak FSFS svnadmin tests.
>>
>> * subversion/tests/cmdline/svntest/main.py
>>    (ensure_list): Converting strings and bytes to lists is a special case.
>>                   Before that, each "element" in them would become a separate
>>                   list element.
>>
>> Modified:
>>      subversion/trunk/subversion/tests/cmdline/svntest/main.py
>>
>> Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
>> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1743195&r1=1743194&r2=1743195&view=diff
>> ==============================================================================
>> --- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
>> +++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Tue May 10 14:29:56 2016
>> @@ -379,6 +379,8 @@ def ensure_list(item):
>>     "If ITEM is not already a list, convert it to a list."
>>     if isinstance(item, list):
>>       return item
>> +  elif isinstance(item, bytes) or isinstance(item, str):
>> +    return [ item ]
>>     else:
>>       return list(item)
>
> I think this is overkill and possibly wrong (and may have been wrong
> before, too); the code should probably just be:
>
>      if isinstance(item, list):
>        return item
>      else:
>        return [item]
>
>
> There's no reason to have a third option, unless you also want to
> convert tuples to lists — but in that case I'd prefer to have an
> explicit check for 'isinstance(item, tuple)' instead of explicit checks
> for strings and byte sequences.
The list construction is there to convert from iterators
(e.g. map, filter) to "proper" lists in Python 3. The strings
special case is mainly carried over from various local
incarnations of that logic.

Is there a base class for iterators which is also known
to Python 2 (doesn't need to be used there) ?

-- Stefan^2.