merging some bugfixes #163

Злито
sarah злито 1 комітів з bigmerge до master 2018-11-22 01:05:24 +00:00
erinn прокоментував(ла) 2018-11-22 00:48:41 +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
buildbot прокоментував(ла) 2018-11-22 00:49:00 +00:00
Учасник
Drone Build Status: failure https://build.openprivacy.ca/cwtch.im/cwtch/278
buildbot прокоментував(ла) 2018-11-22 01:03:26 +00:00
Учасник
Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch/280
sarah закрив цей запит на злиття 2018-11-22 01:05:23 +00:00
dan прокоментував(ла) 2018-11-22 05:43:33 +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
dan прокоментував(ла) 2018-11-22 06:52:22 +00:00
Власник

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
Підпишіться щоб приєднатися до обговорення.
Немає рецензентів
Етап відсутній
Немає виконавця
3 учасників
Сповіщення
Дата завершення
Термін дії не дійсний або знаходиться за межами допустимого діапазону. Будь ласка використовуйте формат 'yyyy-mm-dd'.

Термін виконання не встановлений.

Залежності

No dependencies set.

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