[ cobertura-Patches-2722288 ] Fix LineData so that it can be used during across threads

[SourceForge.net <no...@so...>]

Patches item #2722288, was opened at 2009-03-30 13:08
Message generated for change (Comment added) made by lewijw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=720017&aid=2722288&group_id=130558

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: GS FitNesse (gs-fitnesse)
>Assigned to: John Lewis (lewijw)
Summary: Fix LineData so that it can be used during across threads

Initial Comment:
Fix LineData so that it can be used during across threads.
Fixed bugs registered like this.
http://www.nabble.com/Cobertura-1.9-potential-threading-bug-fix-td19326045.html

----------------------------------------------------------------------

>Comment By: John Lewis (lewijw)
Date: 2009-06-25 06:05

Message:
Thanks for submitting the patch.   As Piotr has commented, this issue was
fixed on June 5th with revision 602.   I hope to release 1.9.2 today.

I was concerned with a possible deadlock during merges and equals, so I
ended up using the java concurrent Lock object added with Java 5.  I am
aquiring a lock on both objects involved in the merge/equals before
proceeding.

----------------------------------------------------------------------

Comment By: Piotr Tabor (ptab)
Date: 2009-06-06 02:27

Message:
Having the same problem. 

Seems to be already fixed in trunk (1.9.1 is still corrupted, by trunk
seems to be ok):

        JumpData getJumpData(int jumpNumber)
        {
                lock.lock();
                try
                {
                        if (jumps == null)
                        {
                                jumps = new ArrayList();
                        }
                        if (jumps.size() <= jumpNumber)
                        {
                                for (int i = jumps.size(); i <=
jumpNumber; jumps.add(new JumpData(i++)));
                        }
                        return (JumpData) jumps.get(jumpNumber);
                }
                finally
                {
                        lock.unlock();
                }
        }


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=720017&aid=2722288&group_id=130558