[patch] Actor.link memory leak

On Fri, Oct 15, 2010 at 06:58:23PM +0400, Alexey Ermakov wrote:
> I'd prefer if it made it into Scala 2.8.1 so I wouldn't have to patch
> it.

You're weeks or months too late for 2.8.1.

2.9.0 isn't that far away though...

On 15 October 2010 20:38, Paul Phillips <paulp@improving.org> wrote:

On Fri, Oct 15, 2010 at 06:58:23PM +0400, Alexey Ermakov wrote:
> I'd prefer if it made it into Scala 2.8.1 so I wouldn't have to patch
> it.

You're weeks or months too late for 2.8.1.

--
Paul Phillips      | One way is to make it so simple that there are
Future Perfect     | obviously no deficiencies. And the other way is to make
Empiricist         | it so complicated that there are no obvious deficiencies.
all hip pupils!    |     -- Hoare

--
Kevin Wright

mail / gtalk / msn : kev.lee.wright@gmail.com
pulse / skype: kev.lee.wright
twitter: @thecoda

Looks like the patch has a bug. Line 800+801 should be included within a bracket, right?

797797      mylinks.foreach((linked: AbstractActor) => {  798798        linked.synchronized {  799799          if (!linked.exiting)   800            linked.unlinkFrom(this)  800801            linked.exit(this, exitReason)  801802        }  802803      }) 

On Fri, Oct 15, 2010 at 4:58 PM, Alexey Ermakov <zee@technocore.ru> wrote:

Hi,
Could someone on the scala team have a look at
<https://lampsvn.epfl.ch/trac/scala/ticket/3920>? I've attached a
patch that fixes a memory leak in the actor library (actors aren't
removed from the link list after exiting) and I'd prefer if it made it
into Scala 2.8.1 so I wouldn't have to patch it.