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

try! instead of ? #1

Closed
RCasatta opened this issue Feb 25, 2018 · 2 comments
Closed

try! instead of ? #1

RCasatta opened this issue Feb 25, 2018 · 2 comments

Comments

@RCasatta
Copy link
Contributor

I see a lot of try! in the code, is there a reason to prefer it over ?
@TheBlueMatt
Copy link
Collaborator

Not that I'm aware of? I just dove in and started writing stuff and didn't change when I discovered "?" existed :p.

@RCasatta
Copy link
Contributor Author

RCasatta commented Mar 1, 2018

submitted PR replacing try! with ? TheBlueMatt#3

@RCasatta RCasatta closed this as completed Mar 1, 2018
TheBlueMatt referenced this issue in TheBlueMatt/rust-lightning Aug 25, 2018
@TheBlueMatt
Update error deserialization in compliance with BOLT #1
6bbeeee
TheBlueMatt referenced this issue in TheBlueMatt/rust-lightning Aug 25, 2018
@TheBlueMatt
Update error deserialization in compliance with BOLT #1
755b76b
@TheBlueMatt TheBlueMatt mentioned this issue Oct 19, 2019
Closed
@TheBlueMatt TheBlueMatt mentioned this issue Nov 26, 2021
Merged
@tnull tnull mentioned this issue May 17, 2022
Merged
@valentinewallace valentinewallace mentioned this issue Jul 15, 2022
Merged
@tnull tnull mentioned this issue May 20, 2023
Merged
ariard added a commit to ariard/rust-lightning that referenced this issue Nov 5, 2023
@ariard
Merge pull request lightningdevkit#1 from ariard/2023-08-update-readm…
d4c39ea 
…e-ln-wizards

Update project documentation to rust-lightning-wizards
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TheBlueMatt @RCasatta