LBRYCommunity/lbcd - Forgejo: Beyond coding. We forge.

LBRYCommunity/lbcd

Fork 0

Commit graph

Author	SHA1	Message	Date
Dave Collins	3b5c2baa2e	blockchain: Consistency pass on subscribe code. This takes care of a few minor nits on the recently merged subscribe code. In particular: - Avoid extra unnecessary allocation on notifications slice - Avoid defer overhead on notification mutex in sendNotifications - Make test function comment start with the name of the function per Effective Go guidelines - Use constant for number of subscribers in test - Don't exceed column 80 in test print	2017-08-15 11:12:56 -05:00
Jim Posen	ca9baf77ec	blockchain: Fix race condition in notifications_test	2017-08-14 22:48:32 -05:00
Jim Posen	19b42c3c9b	Allow multiple subscribers to blockchain notifications. The BlockChain struct emits notifications for various events, but it is only possible to register one listener. This changes the interface and implementations to allow multiple listeners.	2017-08-14 22:03:24 -05:00

Author

SHA1

Message

Date

Dave Collins

3b5c2baa2e

blockchain: Consistency pass on subscribe code.

This takes care of a few minor nits on the recently merged subscribe
code.  In particular:

- Avoid extra unnecessary allocation on notifications slice
- Avoid defer overhead on notification mutex in sendNotifications
- Make test function comment start with the name of the function per
  Effective Go guidelines
- Use constant for number of subscribers in test
- Don't exceed column 80 in test print

2017-08-15 11:12:56 -05:00

Jim Posen

ca9baf77ec

blockchain: Fix race condition in notifications_test

2017-08-14 22:48:32 -05:00

Jim Posen

19b42c3c9b

Allow multiple subscribers to blockchain notifications.

The BlockChain struct emits notifications for various events, but
it is only possible to register one listener. This changes the
interface and implementations to allow multiple listeners.

2017-08-14 22:03:24 -05:00

3 commits