[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #9529 [Tor Check]: Replace check.tp.o webservers with Arlo's Go version
#9529: Replace check.tp.o webservers with Arlo's Go version
-----------------------+----------------------------------------------------
Reporter: arma | Owner:
Type: task | Status: new
Priority: normal | Milestone:
Component: Tor Check | Version:
Keywords: | Parent:
Points: | Actualpoints:
-----------------------+----------------------------------------------------
Comment(by arlolra):
> > - I don't think the mutexes around ExitMap are necessary. The
variable is only written to in LoadLists.
>
> Maps in golang aren't thread safe and LoadLists is signalled to run in
another channel.
On second thought, I was just going on the fact that it's not defined what
happens when you read and write to them simultaneously.
But this is just a pointer swap. You're right, it can be removed.
https://github.com/arlolra/check/blob/master/check.go#L148-L151
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/9529#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs