← Back to team overview

cloud-init-dev team mailing list archive

Re: [Merge] ~rmccabe/cloud-init:bug1705804-2 into cloud-init:master

 

> one comment on ryan's patch inline
> 
> do  not use .split(" "), here is why:
>  > "a b   c".split()
>  ['a', 'b', 'c']
> 
>  > "a b   c".split(" ")
>  ['a', 'b', '', '', 'c']

I had to hit reply to see how those two strings looked different

Even cut and pasting, something is removing the additional spaces you added:

"a b c".split(" ")

Crazy, but point taken,

>>> "a...b.c".split(".")
['a', '', '', 'b', 'c']


-- 
https://code.launchpad.net/~rmccabe/cloud-init/+git/cloud-init/+merge/333722
Your team cloud-init commiters is requested to review the proposed merge of ~rmccabe/cloud-init:bug1705804-2 into cloud-init:master.


References