Wiesz ogólnie to chodziło mi o sens sprawdzania czegoś takiego, bo jak
widzisz jedyna forma sprawdzenia czy moja wizja pokrywa się z Twoją jest
poprzez PR :D Ale jeśli z resztą jest ok to w takim razie dogadam się z
conn_managerem i ustalimy co i jak.


W dniu 9 maja 2014 01:54 użytkownik RoXeon <notifications@github.com>napisał:

> In src/js_injector.coffee:
>
> > +  # @param {Integer} taskId specified task identificator
> > +  # @param {Function} runFun specified task processing function
> > +  # @default {null} runFun
> > +  ###
> > +
> > +  injectCode: (taskId, runFun = null) ->
> > +
> > +    if runFun != null
> > +      console.log "CodeInjector:Loading code for task #{taskId}..."
> > +      taskFunctionsList[taskId] = runFun;
> > +      for client in @connectionManager.getActiveConnections()
> > +        @connectionManager.sendNewTaskToPeer(client, taskId, taskFunctionsList[taskId])
> > +      console.log "CodeInjector::Code loaded for task #{taskId} !"
> > +    else
> > +      console.log "CodeInjectror::Complementing clients for task #{taskId}..."
> > +      if @connectionManager.onCodeLoaded != 'code_loaded'
>
> Ten drugi. Z tego co widzę @connectionManager.onCodeLoaded przyjmuje
> dokładnie jeden argument i jest nim callback który ma być wykonany po
> załadowaniu. Więc ten if nie ma za dużego sensu. Jeśli chodzi o pytanie
> typu "czy jest załadowany" to do tego są funkcje typu
> @connectionManager.getConnectionCapabilities. Swoją drogą też kiedy kod się
> załaduje powinieneś zaznaczać ten fakt przez addConnectionCapabilities. Co
> te funkcje przyjmują do tu już musisz dogadać z autorem ConnectionManagera.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/TheTaz/IO2014/pull/67/files#r12459515>
> .
>