procon_bypass_man 0.1.5 → 0.1.9

data/lib/procon_bypass_man/configuration/layer.rb

@@ -1,13 +1,14 @@
 module ProconBypassMan
   class Configuration
     class Layer
-      attr_accessor :mode, :flips, :macros, :remaps
+      attr_accessor :mode, :flips, :macros, :remaps, :left_analog_stick_caps
 
-      def initialize(mode: :manual, &block)
+      def initialize(mode: :manual)
         self.mode = mode
         self.flips = {}
         self.macros = {}
         self.remaps = {}
+        self.left_analog_stick_caps = {}
         instance_eval(&block) if block_given?
       end
 
@@ -50,7 +51,6 @@ module ProconBypassMan
         end
       end
 
-      PRESET_MACROS = [:fast_return]
       def macro(name, if_pressed: )
         if name.respond_to?(:name)
           macro_name = name.name.to_sym
@@ -72,6 +72,36 @@ module ProconBypassMan
         end
       end
 
+      def left_analog_stick_cap(cap: , if_pressed: nil, force_neutral: nil)
+        hash = { cap: cap }
+
+        case if_pressed
+        when TrueClass
+          raise "not support class"
+        when Symbol, String
+          if_pressed = [if_pressed]
+        when Array, FalseClass
+          # sono mama
+        when NilClass
+          if_pressed = nil
+        else
+          raise "not support if_pressed"
+        end
+
+        if force_neutral
+          case force_neutral
+          when TrueClass, FalseClass
+            raise "ボタンを渡してください"
+          when Symbol, String
+            hash[:force_neutral] = [force_neutral]
+          when Array
+            hash[:force_neutral] = force_neutral
+          end
+        end
+
+        left_analog_stick_caps[if_pressed] = hash
+      end
+
       # @return [Array]
       def flip_buttons
         flips

data/lib/procon_bypass_man/configuration/loader.rb

@@ -1,24 +1,22 @@
 module ProconBypassMan
   class Configuration
     module Loader
+      require 'digest/md5'
+
       def self.load(setting_path: )
-        validation_instance = ProconBypassMan::Configuration.switch_context(:validation) do |instance|
-          begin
-            yaml = YAML.load_file(setting_path) or raise "読み込みに失敗しました"
-            instance.instance_eval(yaml["setting"])
-          rescue SyntaxError
-            instance.errors[:base] << "Rubyのシンタックスエラーです"
-            next(instance)
-          rescue Psych::SyntaxError
-            instance.errors[:base] << "yamlのシンタックスエラーです"
-            next(instance)
+        ProconBypassMan::Configuration.switch_new_context(:validation) do |validation_instance|
+          yaml = YAML.load_file(setting_path) or raise "読み込みに失敗しました"
+          validation_instance.instance_eval(yaml["setting"])
+          validator = Validator.new(validation_instance)
+          if validator.valid?
+            next
+          else
+            raise ProconBypassMan::CouldNotLoadConfigError, validator.errors
           end
-          instance.valid?
-          next(instance)
-        end
-
-        if !validation_instance.errors.empty?
-          raise ProconBypassMan::CouldNotLoadConfigError, validation_instance.errors
+        rescue SyntaxError
+          raise ProconBypassMan::CouldNotLoadConfigError, "Rubyのシンタックスエラーです"
+        rescue Psych::SyntaxError
+          raise ProconBypassMan::CouldNotLoadConfigError, "yamlのシンタックスエラーです"
         end
 
         yaml = YAML.load_file(setting_path)
@@ -33,6 +31,9 @@ module ProconBypassMan
           ProconBypassMan.logger.warn "不明なバージョンです。failoverします"
           ProconBypassMan::Configuration.instance.instance_eval(yaml["setting"])
         end
+
+        File.write(ProconBypassMan.digest_path, Digest::MD5.hexdigest(yaml["setting"]))
+
         ProconBypassMan::Configuration.instance
       end
 

data/lib/procon_bypass_man/configuration/validator.rb

@@ -1,6 +1,11 @@
 module ProconBypassMan
   class Configuration
-    module Validator
+    class Validator
+      def initialize(config)
+        @layers = config.layers
+        @prefix_keys = config.prefix_keys
+      end
+
       # @return [Boolean]
       def valid?
         @errors = Hash.new {|h,k| h[k] = [] }
@@ -38,7 +43,7 @@ module ProconBypassMan
       end
 
       def validate_require_prefix_keys
-        if prefix_keys.empty?
+        if @prefix_keys.empty?
           @errors[:prefix_keys] ||= []
           @errors[:prefix_keys] << "prefix_keys_for_changing_layerに値が入っていません。"
         end
@@ -48,24 +53,20 @@ module ProconBypassMan
         @layers.each do |layer_key, value|
           unverified_buttons = []
           # teplevel target button
-          value.instance_eval { @flips.keys }.map(&:to_sym).each { |b| unverified_buttons << b }
-          value.instance_eval { @remaps.keys }.map(&:to_sym).each { |b| unverified_buttons << b }
+          value.flips.keys.map(&:to_sym).each { |b| unverified_buttons << b }
+          value.remaps.keys.map(&:to_sym).each { |b| unverified_buttons << b }
           # internal target button
-          value.instance_eval {
-            @flips.flat_map { |flip_button, flip_option|
-              flip_option.flat_map { |flip_option_key, flip_option_target_button|
-                next if flip_option_key == :flip_interval
-                next if flip_option_target_button.is_a?(FalseClass) || flip_option_target_button.is_a?(TrueClass)
-                flip_option_target_button
-              }
+          value.flips.flat_map { |_flip_button, flip_option|
+            flip_option.flat_map { |flip_option_key, flip_option_target_button|
+              next if flip_option_key == :flip_interval
+              next if flip_option_target_button.is_a?(FalseClass) || flip_option_target_button.is_a?(TrueClass)
+              flip_option_target_button
             }
           }.compact.each { |b| unverified_buttons << b }
-          value.instance_eval {
-            @remaps.flat_map { |button, option|
-              option.flat_map { |flip_option_key, flip_option_target_button|
-                next if flip_option_target_button.is_a?(FalseClass) || flip_option_target_button.is_a?(TrueClass)
-                flip_option_target_button
-              }
+          value.remaps.flat_map { |_button, option|
+            option.flat_map { |_flip_option_key, flip_option_target_button|
+              next if flip_option_target_button.is_a?(FalseClass) || flip_option_target_button.is_a?(TrueClass)
+              flip_option_target_button
             }
           }.compact.each { |b| unverified_buttons << b }
           unless(nunsupport_buttons = (unverified_buttons - ProconBypassMan::Procon::ButtonCollection::BUTTONS)).length.zero?
@@ -78,8 +79,8 @@ module ProconBypassMan
         @layers.each do |layer_key, value|
           flip_buttons = []
           remap_buttons = []
-          value.instance_eval { @flips.keys }.map(&:to_sym).each { |b| flip_buttons << b }
-          value.instance_eval { @remaps.keys }.map(&:to_sym).each { |b| remap_buttons << b }
+          value.flips.keys.map(&:to_sym).each { |b| flip_buttons << b }
+          value.remaps.keys.map(&:to_sym).each { |b| remap_buttons << b }
           if(duplicated_buttons = flip_buttons & remap_buttons).length > 0
             @errors[:layers] << "レイヤー#{layer_key}で、連打とリマップの定義が重複しているボタン#{duplicated_buttons.join(", ")}があります"
           end

data/lib/procon_bypass_man/configuration.rb

@@ -4,14 +4,13 @@ require "procon_bypass_man/configuration/layer"
 
 module ProconBypassMan
   class Configuration
-    include Validator
-
     attr_accessor :layers,
       :setting_path,
       :mode_plugins,
       :macro_plugins,
       :context,
-      :current_context_key
+      :current_context_key,
+      :neutral_position
 
     def self.instance
       @@current_context_key ||= :main
@@ -19,13 +18,12 @@ module ProconBypassMan
       @@context[@@current_context_key] ||= new
     end
 
-    def self.switch_context(key)
-      @@context[key] ||= new
+    def self.switch_new_context(key)
+      @@context[key] = new
       previous_key = @@current_context_key
       if block_given?
         @@current_context_key = key
         value = yield(@@context[key])
-        @@context[key].reset!
         @@current_context_key = previous_key
         return value
       else
@@ -37,13 +35,21 @@ module ProconBypassMan
       reset!
     end
 
-    MODES = [:manual]
-    def layer(direction, mode: :manual, &block)
-      if mode.respond_to?(:name)
-        mode_name = mode.name.to_sym
-      else
-        mode_name = mode
+    module ManualMode
+      def self.name
+        'manual'
       end
+    end
+    MODES = [:manual]
+    def layer(direction, mode: ManualMode, &block)
+      mode_name = case mode
+                  when String
+                    mode.to_sym
+                  when Symbol
+                    mode
+                  else
+                    mode.name.to_sym
+                  end
       unless (MODES + ProconBypassMan::Procon::ModeRegistry.plugins.keys).include?(mode_name)
         raise("#{mode_name} mode is unknown")
       end
@@ -69,6 +75,11 @@ module ProconBypassMan
       self
     end
 
+    def set_neutral_position(x, y)
+      self.neutral_position = AnalogStickPosition.new(x: x, y: y)
+      self
+    end
+
     def prefix_keys
       @prefix_keys_for_changing_layer
     end
@@ -83,6 +94,7 @@ module ProconBypassMan
         left: Layer.new,
         right: Layer.new,
       }
+      @neutral_position = AnalogStickPosition.new(x: 2124, y: 1808)
     end
   end
 end

data/lib/procon_bypass_man/device_connector.rb

@@ -12,24 +12,7 @@ class ProconBypassMan::DeviceConnector
   PROCON_PATH = "/dev/hidraw0"
   PROCON2_PATH = "/dev/hidraw1"
 
-  # 画面で再接続ができたが状況は変わらない
-  def self.reset_connection!
-    s = new
-    s.add([
-      ["0000"],
-      ["0000"],
-      ["8005"],
-      ["0000"],
-      ["8001"],
-    ], read_from: :switch)
-    s.drain_all
-    s.read_procon
-    s.write_switch("213c910080005db7723d48720a800300000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000")
-    sleep(10) # いらないかも
-    s
-  end
-
-  def self.connect(throw_error_if_timeout: true, enable_at_exit: false)
+  def self.connect
     s = new(throw_error_if_timeout: true, enable_at_exit: false)
     s.add([
       ["0000"],
@@ -37,16 +20,16 @@ class ProconBypassMan::DeviceConnector
       ["8005"],
       ["0010"],
     ], read_from: :switch)
-    # 1
+    # 1. Sends current connection status, and if the Joy-Con are connected,
     s.add([["8001"]], read_from: :switch)
     s.add([/^8101/], read_from: :procon) # <<< 81010003176d96e7a5480000000, macaddressとコントローラー番号を返す
-    # 2
+    # 2. Sends handshaking packets over UART to the Joy-Con or Pro Controller Broadcom chip. This command can only be called once per session.
     s.add([["8002"]], read_from: :switch)
     s.add([/^8102/], read_from: :procon)
     # 3
     s.add([/^0100/], read_from: :switch)
     s.add([/^21/], read_from: :procon)
-    # 4
+    # 4. Forces the Joy-Con or Pro Controller to only talk over USB HID without any timeouts. This is required for the Pro Controller to not time out and revert to Bluetooth.
     s.add([["8004"]], read_from: :switch)
     s.drain_all
     return [s.switch, s.procon]
@@ -75,7 +58,7 @@ class ProconBypassMan::DeviceConnector
         timer = ProconBypassMan::Timer.new
         begin
           timer.throw_if_timeout!
-          data = from_device(item).read_nonblock(128)
+          data = from_device(item).read_nonblock(64)
         rescue IO::EAGAINWaitReadable
           retry
         end
@@ -128,7 +111,7 @@ class ProconBypassMan::DeviceConnector
     timer = ProconBypassMan::Timer.new
     begin
       timer.throw_if_timeout!
-      data = switch.read_nonblock(128)
+      data = switch.read_nonblock(64)
       ProconBypassMan.logger.debug { " >>> #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
@@ -163,7 +146,7 @@ class ProconBypassMan::DeviceConnector
     timer = ProconBypassMan::Timer.new
     begin
       timer.throw_if_timeout!
-      data = procon.read_nonblock(128)
+      data = procon.read_nonblock(64)
       ProconBypassMan.logger.error " <<< #{data.unpack("H*")})"
     rescue IO::EAGAINWaitReadable
       retry
@@ -184,7 +167,7 @@ class ProconBypassMan::DeviceConnector
     timer = ProconBypassMan::Timer.new
     begin
       timer.throw_if_timeout!
-      data = procon.read_nonblock(128)
+      data = procon.read_nonblock(64)
       ProconBypassMan.logger.debug { " <<< #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
@@ -217,7 +200,7 @@ class ProconBypassMan::DeviceConnector
     timer = ProconBypassMan::Timer.new
     begin
       timer.throw_if_timeout!
-      data = switch.read_nonblock(128)
+      data = switch.read_nonblock(64)
       ProconBypassMan.logger.debug { " >>> #{data.unpack("H*")})" }
     rescue IO::EAGAINWaitReadable
       retry
@@ -281,7 +264,7 @@ class ProconBypassMan::DeviceConnector
 
     file = File.open(path, "w+")
     begin
-      file.read_nonblock(128)
+      file.read_nonblock(64)
     rescue EOFError
       file.close
       return false
@@ -303,12 +286,12 @@ class ProconBypassMan::DeviceConnector
     case
     when is_available_device?(PROCON_PATH)
       ProconBypassMan.logger.info "proconのデバイスファイルは#{PROCON_PATH}を使います"
-      @procon = File.open(PROCON_PATH, "w+")
-      @gadget = File.open('/dev/hidg0', "w+")
+      @procon = File.open(PROCON_PATH, "w+b")
+      @gadget = File.open('/dev/hidg0', "w+b")
     when is_available_device?(PROCON2_PATH)
       ProconBypassMan.logger.info "proconのデバイスファイルは#{PROCON2_PATH}を使います"
-      @procon = File.open(PROCON2_PATH, "w+")
-      @gadget = File.open('/dev/hidg0', "w+")
+      @procon = File.open(PROCON2_PATH, "w+b")
+      @gadget = File.open('/dev/hidg0', "w+b")
     else
       raise "/dev/hidraw0, /dev/hidraw1の両方見つかりませんでした"
     end

data/lib/procon_bypass_man/error_reporter.rb

@@ -0,0 +1,44 @@
+require "net/http"
+require "json"
+
+class ProconBypassMan::ErrorReporter
+  PATH = "/api/error_reports" # POST
+
+  class Client
+    def initialize
+      @server = ProconBypassMan.api_server
+      @hostname = `hostname`.chomp
+    end
+
+    def post(body: )
+      # TODO ここでvalidationする
+      if @server.nil?
+        ProconBypassMan.logger.info('送信先が未設定なのでスキップしました')
+        return
+      end
+
+      uri = URI.parse("#{@server}#{PATH}")
+      http = Net::HTTP.new(uri.host, uri.port)
+      http.use_ssl = uri.scheme === "https"
+      response = http.post(
+        uri.path,
+        { report: body.full_message.to_json, hostname: @hostname }.to_json,
+        { "Content-Type" => "application/json" },
+      )
+      unless response.code == /^20/
+        ProconBypassMan.logger.error(response.body)
+      end
+    rescue => e
+      puts e
+      ProconBypassMan.logger.error(e)
+    end
+  end
+
+  def self.report(body: )
+    ProconBypassMan.logger.error(body)
+    Client.new.post(body: body)
+  rescue => e
+    ProconBypassMan.logger.error(e)
+  end
+end
+

data/lib/procon_bypass_man/io_monitor.rb

@@ -62,10 +62,13 @@ module ProconBypassMan
           }.join(", ")
           max_output_length = line.length
           sleep 0.7
-          print "\r"
-          print " " * max_output_length
-          print "\r"
-          print line
+
+          if ENV["PBM_FOREGROUND"]
+            print "\r"
+            print " " * max_output_length
+            print "\r"
+            print line
+          end
           ProconBypassMan.logger.debug { line }
           break if $will_terminate_token
         end

data/lib/procon_bypass_man/on_memory_cache.rb

@@ -0,0 +1,34 @@
+class ProconBypassMan::OnMemoryCache
+  class CacheValue
+    attr_accessor :expired_at, :value
+
+    def initialize(expired_at: , value: )
+      self.expired_at = expired_at
+      self.value = value
+    end
+  end
+
+  def initialize
+    @table = {}
+  end
+
+  # @param [Integer] expires_in 秒数
+  # @param [String] key
+  def fetch(key: , expires_in: , &block)
+    now = Time.now
+    if @table[key].nil?
+      value = block.call
+      value_object = CacheValue.new(expired_at: now + expires_in, value: value)
+      @table[key] = value_object
+      return value
+    end
+
+    if @table[key].expired_at < now
+      value = block.call
+      @table[key] = CacheValue.new(expired_at: now + expires_in, value: value)
+      return value
+    else
+      return @table[key].value
+    end
+  end
+end

data/lib/procon_bypass_man/procon/analog_stick_cap.rb

@@ -0,0 +1,88 @@
+class ProconBypassMan::Procon::AnalogStickCap
+  class Position
+    attr_accessor :x, :y
+
+    def initialize(x:, y:)
+      @x = x.to_i
+      @y = y.to_i
+    end
+
+    def to_binary
+      analog_stick_data = [
+        (@x & "0xff".to_i(16)),
+        ((@y << 4) & "0xf0".to_i(16)) | ((@x >> 8) & "0x0f".to_i(16)),
+        (@y >> 4) & "0xff".to_i(16),
+      ]
+      hex = analog_stick_data.map{ |x| x.to_s(16).rjust(2, "0") }.join
+      [hex].pack("H*")
+    end
+  end
+
+  attr_accessor :bin_x, :bin_y
+  attr_accessor :neutral_position
+
+  def initialize(binary)
+    @neutral_position = ProconBypassMan::Configuration.instance.neutral_position
+    @binary = binary
+
+    byte6 = binary[6].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte7 = binary[7].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte8 = binary[8].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+
+    self.bin_x = "#{byte7[4..7]}#{byte6}"
+    self.bin_y = "#{byte8}#{byte7[0..3]}"
+  end
+
+  # @return [ProconBypassMan::Procon::AnalogStickCap::Position]
+  def capped_position(cap_hypotenuse: )
+    if hypotenuse > cap_hypotenuse
+      relative_capped_x = cap_hypotenuse * Math.cos(rad * Math::PI / 180).abs
+      relative_capped_y = cap_hypotenuse * Math.sin(rad * Math::PI / 180).abs
+      relative_capped_x = -(relative_capped_x.abs) if relative_x.negative?
+      relative_capped_y = -(relative_capped_y.abs) if relative_y.negative?
+      return Position.new(
+        x: relative_capped_x + neutral_position.x,
+        y: relative_capped_y + neutral_position.y,
+      )
+    else
+      return position
+    end
+  end
+
+  # @return [ProconBypassMan::Procon::AnalogStickCap::Position]
+  def position
+    Position.new(x: abs_x, y: abs_y)
+  end
+
+  # 0, 0からのx
+  def abs_x
+    bin_x.to_i(2)
+  end
+
+  # 0, 0からのy
+  def abs_y
+    bin_y.to_i(2)
+  end
+
+  def relative_x
+    bin_x.to_i(2) - neutral_position.x
+  end
+
+  def relative_y
+    bin_y.to_i(2) - neutral_position.y
+  end
+
+  # @deprecated
+  def x; relative_x; end
+  def y; relative_y; end
+
+  def rad
+    (
+      Math.atan(y / x.to_f) * 180 / Math::PI
+    ).floor(6)
+  end
+
+  def hypotenuse
+    Math.sqrt(x**2 + y**2).floor(6)
+  end
+end

data/lib/procon_bypass_man/procon/button_collection.rb

@@ -8,15 +8,20 @@ class ProconBypassMan::Procon::ButtonCollection
     end
   end
 
+  # https://github.com/dekuNukem/Nintendo_Switch_Reverse_Engineering/blob/ac8093c84194b3232acb675ac1accce9bcb456a3/bluetooth_hid_notes.md
+  #0) Input report ID
+  #1) Timer. Increments very fast. Can be used to estimate excess Bluetooth latency.
+  #2 high nibble) Battery level. 8=full, 6=medium, 4=low, 2=critical, 0=empty. LSB=Charging.
+  #2 low nibble) Connection info. (con_info >> 1) & 3 - 3=JC, 0=Pro/ChrGrip. con_info & 1 - 1=Switch/USB powered.
   #3)  ZR	R	SR(right)	SL(right)	A	B	X	Y
   #4)  Grip	(none)	Cap	Home	ThumbL	ThumbR	+	-
   #5)  ZL	L	SL(left)	SR(left)	Left	Right	Up	Down
-  #6)  analog[0]
-  #7)  analog[1]
-  #8)  analog[2]
-  #9)  analog[3]
-  #a)  analog[4]
-  #b)  analog[5]
+  #6)  analog[0] Left analog stick data
+  #7)  analog[1] Left analog stick data
+  #8)  analog[2] Left analog stick data
+  #9)  analog[3] Right analog stick data
+  #a)  analog[4] Right analog stick data
+  #b)  analog[5] Right analog stick data
   BYTES_MAP = {
     0 => nil,
     1 => nil,
@@ -24,6 +29,9 @@ class ProconBypassMan::Procon::ButtonCollection
     3 => [:zr, :r, :sr, :sl, :a, :b, :x, :y],
     4 => [:grip, :_undefined_key, :cap, :home, :thumbl, :thumbr, :plus, :minus],
     5 => [:zl, :l, :sl, :sr, :left, :right, :up, :down],
+    6 => [],
+    7 => [],
+    8 => [],
   }.freeze
 
   BUTTONS_MAP = BYTES_MAP.reduce({}) { |acc, value|

data/lib/procon_bypass_man/procon/debug_dumper.rb

@@ -0,0 +1,17 @@
+class ProconBypassMan::Procon::DebugDumper
+  def initialize(binary: )
+    @binary = binary
+    # ProconBypassMan.logger.debug { "<<< patched #{@binary.unpack("H*")}" }
+  end
+
+  def dump_analog_sticks
+    byte6 = @binary[6].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte7 = @binary[7].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+    byte8 = @binary[8].unpack("H*").first.to_i(16).to_s(2).rjust(8, "0")
+
+    x = "#{byte7[4..7]}#{byte6}"
+    y = "#{byte8}#{byte7[0..3]}"
+    ProconBypassMan.logger.debug "x: #{x}, val: #{x.to_i(2)}"
+    ProconBypassMan.logger.debug "y: #{y}, val: #{y.to_i(2)}"
+  end
+end

data/lib/procon_bypass_man/procon/mode_registry.rb

@@ -23,10 +23,10 @@ class ProconBypassMan::Procon::ModeRegistry
   }
 
   def self.install_plugin(klass)
-    if plugins[klass.name]
+    if plugins[klass.name.to_sym]
       raise "すでに登録済みです"
     end
-    plugins[klass.name] = klass.binaries
+    plugins[klass.name.to_sym] = klass.binaries
   end
 
   def self.load(name)

data/lib/procon_bypass_man/procon/press_button_aware.rb

@@ -0,0 +1,13 @@
+class ProconBypassMan::PpressButtonAware
+  def initialize(binary)
+    @binary = binary
+  end
+
+  def pressed_button?(button)
+    @binary[
+      ::ProconBypassMan::Procon::ButtonCollection.load(button).byte_position
+    ].unpack("H*").first.to_i(16).to_s(2).reverse[
+      ::ProconBypassMan::Procon::ButtonCollection.load(button).bit_position
+    ] == '1'
+  end
+end

data/lib/procon_bypass_man/procon/pressed_button_helper.rb

@@ -1,19 +1,9 @@
 module ProconBypassMan::Procon::PushedButtonHelper
-  module Static
-    def pressed_button?(button)
-      binary[
-        ::ProconBypassMan::Procon::ButtonCollection.load(button).byte_position
-      ].unpack("H*").first.to_i(16).to_s(2).reverse[
-        ::ProconBypassMan::Procon::ButtonCollection.load(button).bit_position
-      ] == '1'
-    end
-  end
-
   module Dynamic
     @@compiled = false
     def compile_if_not_compile_yet!
       unless @@compiled
-        ::ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP.each do |button, value|
+        ::ProconBypassMan::Procon::ButtonCollection::BUTTONS_MAP.each do |button, _value|
           define_method "pressed_#{button}?" do
             pressed_button?(button)
           end