This page is no longer maintained — Please continue to the home page at www.scala-lang.org

[patch] Actor.link memory leak

3 replies
Alexey Ermakov
Joined: 2010-09-06,
User offline. Last seen 42 years 45 weeks ago.

Hi,
Could someone on the scala team have a look at
? 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.

extempore
Joined: 2008-12-17,
User offline. Last seen 35 weeks 3 days ago.
Re: [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.

Kevin Wright 2
Joined: 2010-05-30,
User offline. Last seen 26 weeks 4 days ago.
Re: [patch] Actor.link memory leak
2.9.0 isn't that far away though...

On 15 October 2010 20:38, Paul Phillips <paulp [at] improving [dot] 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 [dot] lee [dot] wright [at] gmail [dot] com
pulse / skype: kev.lee.wright
twitter: @thecoda

tolsen77
Joined: 2008-10-08,
User offline. Last seen 1 year 38 weeks ago.
Re: [patch] Actor.link memory leak
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 [at] technocore [dot] 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.

Copyright © 2012 École Polytechnique Fédérale de Lausanne (EPFL), Lausanne, Switzerland