Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sync some times failed #363

Closed
Xudong-Huang opened this issue Aug 13, 2018 · 2 comments
Closed

sync some times failed #363

Xudong-Huang opened this issue Aug 13, 2018 · 2 comments
Assignees
@qiangshou007
Labels
A-wallet Area: wallet bug Something isn't working
Projects
Wallet client SDK
Milestone
x86_64 Wallet Client

Comments

@Xudong-Huang
Copy link
Contributor

sometimes report no entry found after send a payment
@Xudong-Huang Xudong-Huang added bug Something isn't working A-wallet Area: wallet labels Aug 13, 2018
@Xudong-Huang Xudong-Huang added this to the x86_64 Wallet Client milestone Aug 13, 2018
@Xudong-Huang Xudong-Huang added this to To do in Wallet client SDK via automation Aug 13, 2018
@qiangshou007 qiangshou007 moved this from To do to In progress in Wallet client SDK Aug 13, 2018
@GTnull
Copy link
Contributor

GTnull commented Aug 13, 2018

What you've got is actually expected behavior. Currently the IndexMut on HashMap (which the []= syntax is using) looks up the key first and then sets the value. In this case you're seeing the panic because the lookup is failing due to the key not being present. For insertion you'll currently want to use .insert instead of the overloaded syntax.

@GTnull
Copy link
Contributor

GTnull commented Aug 13, 2018

someone has been occur this bug
rust-lang/cargo#1175

qiangshou007 added a commit that referenced this issue Aug 13, 2018
@qiangshou007
📝 fix sync bug (#363)
d82ab1b
Xudong-Huang pushed a commit that referenced this issue Aug 13, 2018
@qiangshou007 @Xudong-Huang
🐛 fix sync no entry found issue (#363)
6ac11bd 
pull request #368
Wallet client SDK automation moved this from In progress to Done Aug 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qiangshou007 qiangshou007

Labels
A-wallet Area: wallet bug Something isn't working
Projects
Wallet client SDK
  
Done
Milestone
x86_64 Wallet Client
Development

No branches or pull requests
3 participants
@Xudong-Huang @GTnull @qiangshou007