fix: return @@context even when the unless block is not executed. #4
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.
When I run the example script, the following output is encountered:
I investigated and found that the @@context pointer is populated and returned in the unless block within the
self.context
method the first time it is invoked, but the second timeself.context
is invoked, nothing is returned, and the resultingNFC.context
would be a null pointer, causing the error above.This PR is to add one line in
self.context
to return @@context regardless of whether it has just been assigned or it has been assigned previously.Versions:
Ruby: ruby 2.4.0p0 (2016-12-24 revision 57164) [armv7l-linux-eabihf] (I am running this in a Raspberry Pi 3 Model B)
LibFreeFare: libfreefare-0.4.0 (compiled from source)
Libnfc: libnfc-1.7.1 (compiled from source)