Loop() is not goroutine safe

[Joshua-Anderson]

Package tests run a second connection with go irccon.Loop(), like here.

I've been using this code in my own libraries with -race and discovered a bunch of race conditions, such as github.com/thoj/go-ircevent.(*Connection).RunCallbacks() and github.com/thoj/go-ircevent.(*Connection).AddCallback()

These issues seem like they could easily be solved by adding some mutexes around unsafe operations like callback map access. I may have the time to fix this in the next few weeks, but I'm not sure.

[thoj]

Work in progress. Latest change will fix some of the issues. See da78ed5

[thoj]

Most races has been fixed.