Relocated attach_function in Ext module back into engine to resolve gem complication issues

Author: johanns

lib/scrypt.rb

@@ -2,8 +2,9 @@
 
 # A wrapper for the scrypt algorithm.
 
-require 'scrypt/engine'
 require 'scrypt/errors'
-require 'scrypt/password'
 require 'scrypt/scrypt_ext'
 require 'scrypt/security_utils'
+
+require 'scrypt/engine'
+require 'scrypt/password'

lib/scrypt/engine.rb

@@ -4,7 +4,23 @@
 require 'openssl'
 
 module SCrypt
+  module Ext
+    # rubocop:disable Style/SymbolArray
+    # Bind the external functions
+    attach_function :sc_calibrate,
+                    [:size_t, :double, :double, :pointer],
+                    :int,
+                    blocking: true
+
+    attach_function :crypto_scrypt,
+                    [:pointer, :size_t, :pointer, :size_t, :uint64, :uint32, :uint32, :pointer, :size_t],
+                    :int,
+                    blocking: true # todo
+    # rubocop:enable
+  end
+
   class Engine
+    # rubocop:disable Style/MutableConstant
     DEFAULTS = {
       key_len: 32,
       salt_size: 32,
@@ -13,6 +29,7 @@ class Engine
       max_time: 0.2,
       cost: nil
     }
+    # rubocop:enable
 
     class Calibration < FFI::Struct
       layout  :n, :uint64,

lib/scrypt/scrypt_ext.rb

@@ -8,20 +8,5 @@ module Ext
     extend FFI::Library
 
     ffi_lib FFI::Compiler::Loader.find('scrypt_ext')
-
-    # rubocop:disable Style/SymbolArray
-
-    # Bind the external functions
-    attach_function :sc_calibrate,
-                    [:size_t, :double, :double, :pointer],
-                    :int,
-                    blocking: true
-
-    attach_function :crypto_scrypt,
-                    [:pointer, :size_t, :pointer, :size_t, :uint64, :uint32, :uint32, :pointer, :size_t],
-                    :int,
-                    blocking: true # todo
-    
-    # rubocop:enable
   end
 end