[Jython-checkins] SF.net SVN: jython:[6929] trunk/jython/src/org/python/core/PyDictionary. java

[<pj...@us...>]

Revision: 6929
          http://jython.svn.sourceforge.net/jython/?rev=6929&view=rev
Author:   pjenvey
Date:     2009-10-29 04:28:03 +0000 (Thu, 29 Oct 2009)

Log Message:
-----------
avoid unnecessary copying when creating lists

Modified Paths:
--------------
    trunk/jython/src/org/python/core/PyDictionary.java

Modified: trunk/jython/src/org/python/core/PyDictionary.java
===================================================================
--- trunk/jython/src/org/python/core/PyDictionary.java	2009-10-29 04:26:36 UTC (rev 6928)
+++ trunk/jython/src/org/python/core/PyDictionary.java	2009-10-29 04:28:03 UTC (rev 6929)
@@ -619,7 +619,7 @@
         for (Entry<PyObject, PyObject> entry : table.entrySet()) {
             list.add(new PyTuple(entry.getKey(), entry.getValue()));
         }
-        return new PyList(list);
+        return PyList.fromList(list);
     }
 
     /**
@@ -631,12 +631,12 @@
 
     @ExposedMethod(doc = BuiltinDocs.dict_keys_doc)
     final PyList dict_keys() {
-        return new PyList(new ArrayList<PyObject>(table.keySet()));
+        return PyList.fromList(new ArrayList<PyObject>(table.keySet()));
     }
 
     @ExposedMethod(doc = BuiltinDocs.dict_values_doc)
     final PyList dict_values() {
-        return new PyList(new ArrayList<PyObject>(table.values()));
+        return PyList.fromList(new ArrayList<PyObject>(table.values()));
     }
 
     /**


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.