merging some bugfixes #163

병합
sarah bigmerge 에서 master 로 1 commits 를 머지했습니다 2018-11-22 01:05:24 +00:00
소유자

we need to be able to set the AIF before launching individual peers, so this splits that out

we need to be able to set the AIF before launching individual peers, so this splits that out
멤버
Drone Build Status: failure https://build.openprivacy.ca/cwtch.im/cwtch/278
멤버
Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch/280
sarah closed this pull request 2018-11-22 01:05:23 +00:00
소유자

i did away with startPeer in app
Listen now does the go function thing, so you can call peer.Listen directly and itll fork

i did away with startPeer in app Listen now does the go function thing, so you can call peer.Listen directly and itll fork
소유자

so yeah
lines 113-114

 		p.Init(app.mn)
 		p.Listen()

start and launch the peer. I got rid of app.startPeer cus its wrong and terrible

to be fair, LaunchPeers will basically nop
because
.Listen go calls .listenFn inside a look to handle socket fails
and you added cp.started = true inside listenFn
so LaunchPeers should always find them all started already

so yeah lines 113-114 ``` p.Init(app.mn) p.Listen() ``` start and launch the peer. I got rid of app.startPeer cus its wrong and terrible to be fair, LaunchPeers will basically nop because .Listen go calls .listenFn inside a look to handle socket fails and you added `cp.started = true` inside listenFn so LaunchPeers should always find them all `started` already
"로그인하여 이 대화에 참여"
No reviewers
마일스톤 없음
담당자 없음
참여자 3명
알림
마감일
기한이 올바르지 않거나 범위를 벗어났습니다. 'yyyy-mm-dd'형식을 사용해주십시오.

마감일이 설정되지 않았습니다.

의존성

No dependencies set.

Reference: cwtch.im/cwtch#163
No description provided.