Is it necessary to use a mutable Map?



On Sat, Mar 2, 2013 at 10:54 AM, Antonio Salazar Cardozo <
notifications@github.com> wrote:

> In web/webkit/src/main/scala/net/liftweb/http/CssBoundScreen.scala:
>
> > @@ -97,7 +99,7 @@ trait CssBoundScreen extends ScreenWizardRendered with Loggable {
> >
> >    protected def mapLocalAction[T](func: () => JsCmd)(f: String => T): T = {
> >      val name = randomString(20)
> > -    LocalActions.set(LocalActions.is + (name -> func))
> > +    LocalActions.get += (name -> func)
>
> I will let you judge if this affects that, but keep in mind that a slow
> Ajax request will time out on the client, at which point the client is free
> to fire off another request. Exponential backoff means that request may not
> be a retry (a retry won't rerun the code anymore), so you can end up with
> two or more requests running from a given session at a given moment.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/lift/framework/pull/1415/files#r3215252>
> .
>