Merge pull request #565 from undingen/self_assign

Handle list self assignment during slicing

src/runtime/list.cpp

@@ -393,6 +393,9 @@ extern "C" Box* listSetitemSlice(BoxedList* self, BoxedSlice* slice, Box* v) {
     sliceIndex(slice->stop, &stop);
     sliceIndex(slice->step, &step);
 
+    if (self == v) // handle self assignment by creating a copy
+        v = _listSlice(self, 0, self->size, 1, self->size);
+
     if (step != 1) {
         int r = list_ass_ext_slice(self, slice, v);
         if (r)

test/integration/virtualenv_test.py

@@ -21,7 +21,7 @@ set -e
 set -ux
 python -c 'import __future__'
 python -c 'import sys; print sys.executable'
-pip install bcrypt==1.1.0 python-gflags==2.0 sqlalchemy==1.0.0 Pillow==2.8.1 decorator oauth2client
+pip install bcrypt==1.1.0 python-gflags==2.0 sqlalchemy==1.0.0 Pillow==2.8.1 decorator==3.4.2 oauth2client==1.4.11
 python -c 'import bcrypt; assert bcrypt.__version__ == "1.1.0"; assert bcrypt.hashpw("password1", "$2a$12$0123456789012345678901").endswith("I1hdtg4K"); print "bcrypt seems to work"'
 python -c 'import gflags; print "gflags imports"'
 python -c 'import sqlalchemy; print "sqlalchemy imports"'

test/tests/list.py

@@ -5,6 +5,7 @@ l[0] = 1
 print l
 print l[2]
 print l[2L]
+l[:] = l
 
 l = range(5)
 while l: