jython-checkins

[Jython-checkins] SF.net SVN: jython:[6032] trunk/jython/src/org/python/modules

[<fwi...@us...>]

Revision: 6032
          http://jython.svn.sourceforge.net/jython/?rev=6032&view=rev
Author:   fwierzbicki
Date:     2009-02-15 18:22:18 +0000 (Sun, 15 Feb 2009)

Log Message:
-----------
Fixed two infinite recursions found with FindBugs.

Modified Paths:
--------------
    trunk/jython/src/org/python/modules/_csv/PyWriter.java
    trunk/jython/src/org/python/modules/thread/PyLocal.java

Modified: trunk/jython/src/org/python/modules/_csv/PyWriter.java
===================================================================
--- trunk/jython/src/org/python/modules/_csv/PyWriter.java	2009-02-15 14:23:28 UTC (rev 6031)
+++ trunk/jython/src/org/python/modules/_csv/PyWriter.java	2009-02-15 18:22:18 UTC (rev 6032)
@@ -51,7 +51,7 @@
             "elements will be converted to string.");
 
     public void writerows(PyObject seqseq) {
-        writerows(seqseq);
+        writer_writerows(seqseq);
     }
 
     @ExposedMethod

Modified: trunk/jython/src/org/python/modules/thread/PyLocal.java
===================================================================
--- trunk/jython/src/org/python/modules/thread/PyLocal.java	2009-02-15 14:23:28 UTC (rev 6031)
+++ trunk/jython/src/org/python/modules/thread/PyLocal.java	2009-02-15 18:22:18 UTC (rev 6032)
@@ -72,7 +72,7 @@
     @Override
     @ExposedSet(name = "__dict__")
     public void setDict(PyObject dict) {
-        setDict(dict);
+        super.setDict(dict);
     }
 
     @Override


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

[Jython-checkins] SF.net SVN: jython:[6336] trunk/jython/src/org/python/modules

[<zy...@us...>]

Revision: 6336
          http://jython.svn.sourceforge.net/jython/?rev=6336&view=rev
Author:   zyasoft
Date:     2009-05-11 03:48:59 +0000 (Mon, 11 May 2009)

Log Message:
-----------
More findbugs fixes: removed inconsistent synchronization for
cPickle.PickleMemo (should be thread confined anyway).

Modified Paths:
--------------
    trunk/jython/src/org/python/modules/_marshal.java
    trunk/jython/src/org/python/modules/cPickle.java

Modified: trunk/jython/src/org/python/modules/_marshal.java
===================================================================
--- trunk/jython/src/org/python/modules/_marshal.java	2009-05-11 02:39:06 UTC (rev 6335)
+++ trunk/jython/src/org/python/modules/_marshal.java	2009-05-11 03:48:59 UTC (rev 6336)
@@ -416,8 +416,7 @@
                     int size = read_int();
                     String s = read_string(size);
                     if (type == TYPE_INTERNED) {
-                        s.intern(); // do we really honor like this?
-                        PyString pys = PyString.fromInterned(s);
+                        PyString pys = PyString.fromInterned(s.intern());
                         strings.append(pys);
                         return pys;
                     } else {

Modified: trunk/jython/src/org/python/modules/cPickle.java
===================================================================
--- trunk/jython/src/org/python/modules/cPickle.java	2009-05-11 02:39:06 UTC (rev 6335)
+++ trunk/jython/src/org/python/modules/cPickle.java	2009-05-11 03:48:59 UTC (rev 6336)
@@ -1426,7 +1426,7 @@
     /*
      * A very specialized and simplified version of PyStringMap. It can
      * only use integers as keys and stores both an integer and an object
-     * as value. It is very private!
+     * as value. It is very private! And should only be used thread-confined.
      */
     static private class PickleMemo {
         //Table of primes to cycle through
@@ -1456,7 +1456,7 @@
             this(4);
         }
 
-        public synchronized int size() {
+        public int size() {
             return size;
         }
 
@@ -1521,7 +1521,7 @@
         }
 
 
-        private synchronized final void resize(int capacity) {
+        private final void resize(int capacity) {
             int p = prime;
             for(; p<primes.length; p++) {
                 if (primes[p] >= capacity) break;


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