Skip to content

remove singletons#740

Merged
superboyiii merged 5 commits intoneo-project:masterfrom
bettybao1209:sync-neo2302
Feb 11, 2021
Merged

remove singletons#740
superboyiii merged 5 commits intoneo-project:masterfrom
bettybao1209:sync-neo2302

Conversation

@bettybao1209
Copy link
Copy Markdown
Contributor

@bettybao1209 bettybao1209 commented Feb 8, 2021

need to add a new msg type GetInstance in LocalNode class like this:

protected override void OnReceive(object message)
        {
            base.OnReceive(message);
            switch (message)
            {
                case Message msg:
                    BroadcastMessage(msg);
                    break;
                case RelayDirectly relay:
                    OnRelayDirectly(relay.Inventory);
                    break;
                case SendDirectly send:
                    OnSendDirectly(send.Inventory);
                    break;
                case GetInstance _:
                    Sender.Tell(this);
                    break;
            }
        }

@erikzhang
Copy link
Copy Markdown
Member

erikzhang commented Feb 8, 2021

Please check neo-project/neo#2311

@cloud8little
Copy link
Copy Markdown
Contributor

test passed.

@superboyiii superboyiii merged commit 83fe4d1 into neo-project:master Feb 11, 2021
@bettybao1209 bettybao1209 deleted the sync-neo2302 branch February 11, 2021 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants