On Thu, Dec 17, 2009 at 4:35 AM, Vesa <brut...@gmail.com> wrote:

> Hi,
>
> I was trying to run the skittr example from the lift-examples and it
> doesn't seem to work. It's demonstrating usage of comet but it doesn't
> get that far as the comet actor never receives any messages due to a
> bug. Also the indentation is a mix of spaces and tabs of different
> width and looks quite messed up. How should I deliver a fix when I'm
> done?
>

There's a single simple bug.  When we moved from Scala Actors to Lift
Actors, we moved from a case object AddTimelineViewer to case class
AddTimelineViewer(who: SimpleActor[Any])  Unfortunately, the compiler didn't
flag the actor ! AddTimelineViewer line as a bug... it converted the line to
actor ! (who => AddTimelineViewer.apply) and passed a function as a
message.  I've fixed the code locally (WatchUser.scala lines 49-52 become):

  override def localSetup {
    userActor = name.flatMap(name => UserList.find(name))
    userActor.foreach{ua => ua ! AddTimelineViewer(this) ;  messages = (ua
!? GetTimeline) match {case Timeline(m) => m; case _ => Nil}}
  }

I'll get a patch onto review board by end of day today.



>
> - Vesa
>
> --
>
> You received this message because you are subscribed to the Google Groups
> "Lift" group.
> To post to this group, send email to lift...@googlegroups.com.
> To unsubscribe from this group, send email to
> liftweb+unsubscr...@googlegroups.com<liftweb%2bunsubscr...@googlegroups.com>
> .
> For more options, visit this group at
> http://groups.google.com/group/liftweb?hl=en.
>
>
>


-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Surf the harmonics

--

You received this message because you are subscribed to the Google Groups 
"Lift" group.
To post to this group, send email to lift...@googlegroups.com.
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en.


Reply via email to