search

carlosg2 / circulate

Automatically exported from code.google.com/p/circulate
0 stars 0 forks source link

code review on Network and Node #1

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 

so when using a loop on enterframe everything slow down

we need to use events mainly to keep a smooth UI

and also be able to instantiate a shit load of UI

review need to focus on
- the event dispatching from nodes and network
- avoid duplication of events
- particularly look at infinite loop case event -> clientA <--RTMFP-->clientB 
--> event --> etc.
- we need also to review the logic between events and commands
- most important thing is to be able to update all users on the command center

little known problem:

when we duplicate a lot of local clients (100+)
not all clients are connected to each other

post() is not good enough, we really need to fully reuse 
sendToNearest/sendToNeighbor

to be sure that all clients are known on the network

we need for that to implement a sturdy keepalive and send userlist commands

Original issue reported on code.google.com by zwetan on 15 Mar 2013 at 8:29