lang: Remove a potential panic while getting the IDL in declare_program!
#3458
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
A part of the getting the IDL process can panic if getting the environment variable fails:
anchor/lang/attribute/program/src/declare_program/mod.rs
Line 37 in 6ff6655
Although this is unlikely to panic given this is a
cargo
environment variable, it still makes sense to convert this to anErr
variant rather than potentially panicking, especially considering theget_idl
function already returning aResult
type.Summary of changes
Remove a potential panic while getting the IDL in
declare_program!
.