Skip to content

Commit

Permalink
Avoid warning about undefined wariables
Browse files Browse the repository at this point in the history
warning: instance variable @ffi_functions not initialized
  • Loading branch information
larskanis committed Mar 29, 2024
1 parent 0727ada commit 5bfe1a7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion lib/ffi/function.rb
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ def param_types
# On CRuby it also ensures that it does not get garbage collected.
module RegisterAttach
def attach(mod, name)
funcs = mod.instance_variable_get("@ffi_functions")
funcs = mod.instance_variable_defined?("@ffi_functions") && mod.instance_variable_get("@ffi_functions")
unless funcs
funcs = {}
mod.instance_variable_set("@ffi_functions", funcs)
Expand Down
4 changes: 2 additions & 2 deletions lib/ffi/library.rb
Original file line number Diff line number Diff line change
Expand Up @@ -552,10 +552,10 @@ def attached_functions
# @return [Hash< Symbol => ffi_type >]
def attached_variables
(
(@ffi_gsvars || {}).map do |name, gvar|
(defined?(@ffi_gsvars) ? @ffi_gsvars : {}).map do |name, gvar|
[name, gvar.class]
end +
(@ffi_gvars || {}).map do |name, gvar|
(defined?(@ffi_gvars) ? @ffi_gvars : {}).map do |name, gvar|
[name, gvar.layout[:gvar].type]
end
).to_h
Expand Down

0 comments on commit 5bfe1a7

Please sign in to comment.