rubywmq 2.0.2 → 2.1.0

data/examples/q_to_files.rb

@@ -1,7 +1,6 @@
 #
 # Example: q_to_files:  Copy all messages in a queue to separate files in a directory
 #
-require 'rubygems'
 require 'find'
 require 'yaml'
 require 'wmq'
@@ -9,22 +8,22 @@ require 'fileutils'
 
 # Call program passing environment name as first parameter
 #   The environment corresponds to an entry in the config file
-env = ARGV[0] || raise("Command line argument 'environment' is required")
+env    = ARGV[0] || raise("Command line argument 'environment' is required")
 config = YAML::load_file('q_to_files.cfg')[env]
 
 # Create supplied path if it does not exist
-path = config['target_directory']
+path   = config['target_directory']
 FileUtils.mkdir_p(path)
 
-message = WMQ::Message.new
+message            = WMQ::Message.new
 message.descriptor = config['descriptor'] || {}
-tstart = Time.now
-counter = 0
+tstart             = Time.now
+counter            = 0
 WMQ::QueueManager.connect(config['qmgr_options']) do |qmgr|
   qmgr.open_queue(config['input_queue']) do |queue|
     queue.each do |message|
       counter = counter + 1
-      File.open(File.join(path, "message_%03d" % counter), 'w') {|file| file.write(message.data) }
+      File.open(File.join(path, "message_%03d" % counter), 'w') { |file| file.write(message.data) }
     end
   end
 end

data/examples/request.rb

@@ -6,39 +6,41 @@
 #
 #          A temporary Dynamic Reply To Queue is used with non-persistent messaging
 #
-require 'rubygems'
 require 'wmq'
 
 wait_seconds = 30
 
-WMQ::QueueManager.connect(:q_mgr_name=>'REID') do |qmgr|
-  qmgr.open_queue(:q_name        => 'SYSTEM.DEFAULT.MODEL.QUEUE',
-                  :dynamic_q_name=> 'REQUEST.*',
-                  :mode          => :input
-                 ) do |reply_queue|
+WMQ::QueueManager.connect(q_mgr_name: 'REID') do |qmgr|
+  qmgr.open_queue(
+    q_name:         'SYSTEM.DEFAULT.MODEL.QUEUE',
+    dynamic_q_name: 'REQUEST.*',
+    mode:           :input
+  ) do |reply_queue|
 
-    message = WMQ::Message.new()
-    message.descriptor = { :msg_type      => WMQ::MQMT_REQUEST,
-                           :reply_to_q    => reply_queue.name,
-                           :reply_to_q_mgr=> qmgr.name,
-                           :format        => WMQ::MQFMT_STRING,
-                           :expiry        => wait_seconds*10}  # Measured in tenths of a second
-    message.data = 'Hello World'
+    message            = WMQ::Message.new()
+    message.descriptor = {
+      msg_type:       WMQ::MQMT_REQUEST,
+      reply_to_q:     reply_queue.name,
+      reply_to_q_mgr: qmgr.name,
+      format:         WMQ::MQFMT_STRING,
+      expiry:         wait_seconds*10 # Measured in tenths of a second
+    }
+    message.data       = 'Hello World'
 
     # Send request message
-    qmgr.put(:q_name=>'TEST.QUEUE', :message=>message)
+    qmgr.put(q_name: 'TEST.QUEUE', message: message)
 
     # Copy outgoing Message id to correlation id
-    message.descriptor[:correl_id]=message.descriptor[:msg_id]
+    message.descriptor[:correl_id] = message.descriptor[:msg_id]
 
     # Wait for reply
     #   Only get the message that matches the correlation id set above
-    if reply_queue.get(:wait=>wait_seconds*1000, :message=>message, :match=>WMQ::MQMO_MATCH_CORREL_ID)
-      puts "Received:"
+    if reply_queue.get(wait: wait_seconds * 1000, message: message, match: WMQ::MQMO_MATCH_CORREL_ID)
+      puts 'Received:'
       puts message.data
     else
       # get returns false when no message received. Also: message.data = nil
-      puts "Timed Out waiting for a reply from the server"
+      puts 'Timed Out waiting for a reply from the server'
     end
   end
 end

data/examples/server.rb

@@ -3,12 +3,12 @@
 #
 #     This server times out after 60 seconds, but could be modified to
 #     run forever. Example:
-#         queue.each(:wait=>-1) do |message|
+#         queue.each(wait: -1) do |message|
 #
 #     Note: - All calls are being performed under synchpoint control to
 #             prevent messages being if the program terminates unexpectedly
 #             or an error occurrs.
-#             Uses:  :sync=>true
+#             Uses:  sync: true
 #           - Queue#each will backout any message changes if an excecption is raised
 #             but not handled within the each block
 #
@@ -46,31 +46,34 @@
 #                 - E.g. Database, File etc..
 #             - etc....
 #
-require 'rubygems'
 require 'wmq'
 
-WMQ::QueueManager.connect(:q_mgr_name=>'REID') do |qmgr|
-  qmgr.open_queue(:q_name=>'TEST.QUEUE', :mode=>:input) do |queue|
-    queue.each(:wait=>60000, :sync=>true, :convert=>true) do |request|
-      puts "Data Received:"
+WMQ::QueueManager.connect(q_mgr_name: 'REID') do |qmgr|
+  qmgr.open_queue(q_name: 'TEST.QUEUE', mode: :input) do |queue|
+    queue.each(wait: 60000, sync: true, convert: true) do |request|
+      puts 'Data Received:'
       puts request.data
 
       begin
-        reply = WMQ::Message.new
+        reply      = WMQ::Message.new
         reply.data = 'Echo back:'+request.data
 
-        qmgr.put_to_reply_q(:message=>reply,
-                            :request_message=>request, # Only replies if message type is request
-                            :sync=>true)
+        qmgr.put_to_reply_q(
+          message:         reply,
+          request_message: request, # Only replies if message type is request
+          sync:            true
+        )
 
       rescue WMQ::WMQException => exc
         # Failed to send reply, put message to the Dead Letter Queue and add a dead letter header
         p exc
-        puts "Failed to reply to sender, Put to dead letter queue"
-        qmgr.put_to_dead_letter_q(:message=>request,
-                                  :reason=>WMQ::MQRC_UNKNOWN_REMOTE_Q_MGR,
-                                  :q_name=>queue.name,
-                                  :sync=>true)
+        puts 'Failed to reply to sender, Put to dead letter queue'
+        qmgr.put_to_dead_letter_q(
+          message: request,
+          reason:  WMQ::MQRC_UNKNOWN_REMOTE_Q_MGR,
+          q_name:  queue.name,
+          sync:    true
+        )
         # If it fails to put to the dead letter queue, this program will terminate and
         # the changes will be "backed out". E.g. Queue Full, ...
       end

data/ext/extconf.rb

@@ -7,7 +7,8 @@ require 'generate_structs'
 
 include_path = ''
 if RUBY_PLATFORM =~ /win|mingw/i
-  include_path = 'C:\Program Files\IBM\WebSphere MQ\tools\c\include'
+  x86_path     = 'C:\Program Files (x86)\IBM\WebSphere MQ\tools\c\include'
+  include_path = File.directory?(x86_path) ? x86_path : 'C:\Program Files\IBM\WebSphere MQ\tools\c\include'
   dir_config('mqm', include_path, '.')
 else
   include_path = '/opt/mqm/inc'

data/ext/extconf_client.rb

@@ -4,9 +4,9 @@
 #        Windows, Solaris and Linux
 #
 require 'mkmf'
-require '../../generate/generate_reason'
-require '../../generate/generate_const'
-require '../../generate/generate_structs'
+require_relative '../../generate/generate_reason'
+require_relative '../../generate/generate_const'
+require_relative '../../generate/generate_structs'
 
 include_path = ''
 unless (RUBY_PLATFORM =~ /win/i) || (RUBY_PLATFORM =~ /solaris/i) || (RUBY_PLATFORM =~ /linux/i)

data/ext/generate/generate_const.rb

@@ -3,17 +3,22 @@ class GenerateConst
 
   # Extract Constants from Header files
   def GenerateConst.extract_const(filename, const_prefix, start_exp=nil, end_exp=nil)
-    @constants = []
-    active = if start_exp then false else true end    # Sure there is a better way
+    constants = []
+    active     =
+      if start_exp then
+        false
+      else
+        true
+      end # Sure there is a better way
     File.open(filename) do |file|
       file.each do |line|
         line.rstrip!
-        if line.length > 0   # Skip empty lines
+        if line.length > 0 # Skip empty lines
           if active
             break if start_exp && line.match(end_exp)
             # Skip Comment lines, then check for a match
             if (line !~ /^\s*\/\*/) && (match = /\s*#define\s+(#{const_prefix}\w+)[\(\s]+([-\\\w\dx\'\"\s]+)/.match(line))
-              @constants << [match[1], match[2]]
+              constants << [match[1], match[2]]
             end
           else
             next unless line.match(start_exp)
@@ -22,7 +27,7 @@ class GenerateConst
         end
       end
     end
-    @constants
+    constants
   end
 
   # Extract MQ Constants from Header files
@@ -47,8 +52,8 @@ class GenerateConst
   def GenerateConst.config(filename, prefix)
     str = "# WMQ::QueueManager execute commands\n"
     str << "execute_commands:\n"
-    GenerateConst.extract_const(filename,prefix).each do |item|
-      name = item[0].gsub(prefix,'').downcase
+    GenerateConst.extract_const(filename, prefix).each do |item|
+      name = item[0].gsub(prefix, '').downcase
       str << "  :%-26s " % "#{name}:"
       match = name.match(/\w+?_([\w_]+)/)
       str << ":#{match[1]}" if match
@@ -68,28 +73,29 @@ class GenerateConst
 ################################################################################
 module WMQ
 END_OF_STRING
-    [ ['Connect Options','cmqc.h','MQCNO_',/(VERSION)|(CONN_TAG)|(HANDLE_SHARE)|(_LENGTH)/],
-      ['Open Options', 'cmqc.h','MQOO_' ],
-      ['Close Options', 'cmqc.h','MQCO_'],
-      ['Match Options', 'cmqc.h','MQMO_'],
-      ['Message Format Options', 'cmqc.h','MQFMT_',/(_ARRAY)/],
-      ['Get Message Options', 'cmqc.h','MQGMO_',/(VERSION)|(LENGTH)|(MQGMO_BROWSE_HANDLE)|(MQGMO_BROWSE_CO_OP)/],
-      ['Transport Types', 'cmqxc.h','MQXPT_'] ,
+    [
+      ['Connect Options', 'cmqc.h', 'MQCNO_', /(VERSION)|(CONN_TAG)|(HANDLE_SHARE)|(_LENGTH)/],
+      ['Open Options', 'cmqc.h', 'MQOO_'],
+      ['Close Options', 'cmqc.h', 'MQCO_'],
+      ['Match Options', 'cmqc.h', 'MQMO_'],
+      ['Message Format Options', 'cmqc.h', 'MQFMT_', /(_ARRAY)/],
+      ['Get Message Options', 'cmqc.h', 'MQGMO_', /(VERSION)|(LENGTH)|(MQGMO_BROWSE_HANDLE)|(MQGMO_BROWSE_CO_OP)/],
+      ['Transport Types', 'cmqxc.h', 'MQXPT_'],
       ['Report Options', 'cmqc.h', 'MQRO_'],
       ['Message Types', 'cmqc.h', 'MQMT_'],
       ['Expiry', 'cmqc.h', 'MQEI_'],
       ['Feedback Values', 'cmqc.h', 'MQFB_'],
-      ['Encoding Values', 'cmqc.h', 'MQENC_',/(MQENC_NORMAL)|(MQENC_REVERSED)|(MQENC_S390)|(MQENC_TNS)/],
+      ['Encoding Values', 'cmqc.h', 'MQENC_', /(MQENC_NORMAL)|(MQENC_REVERSED)|(MQENC_S390)|(MQENC_TNS)/],
       ['Coded Character Set Identifiers', 'cmqc.h', 'MQCCSI_'],
       ['Priority', 'cmqc.h', 'MQPRI_'],
       ['Persistence', 'cmqc.h', 'MQPER_'],
       ['Put Application Types', 'cmqc.h', 'MQAT_'],
       ['Message Flags', 'cmqc.h', 'MQMF_'],
       ['Original Length', 'cmqc.h', 'MQOL_'],
-      ['Put Message Options', 'cmqc.h', 'MQPMO_',/(VERSION)|(LENGTH)/],
-      ['Put Message Record Fields', 'cmqc.h', 'MQPMRF_',/(VERSION)|(LENGTH)/],
-      ['Reason Codes', 'cmqc.h','MQRC_'],
-      ].each do |item|
+      ['Put Message Options', 'cmqc.h', 'MQPMO_', /(VERSION)|(LENGTH)/],
+      ['Put Message Record Fields', 'cmqc.h', 'MQPMRF_', /(VERSION)|(LENGTH)/],
+      ['Reason Codes', 'cmqc.h', 'MQRC_'],
+    ].each do |item|
       str << "\n# #{item[0]}\n"
       str << GenerateConst.rb_const("#{path}/#{item[1]}", item[2], item[3])
     end
@@ -108,10 +114,11 @@ END_OF_STRING
 module WMQ
 END_OF_STRING
     str << "# Admin constants from cmqc.h\n"
-    [ ['Object Types', 'cmqc.h', 'MQOT_'],
+    [
+      ['Object Types', 'cmqc.h', 'MQOT_'],
       ['Security Identifier Types', 'cmqc.h', 'MQSIDT_'],
       ['Channel Types', 'cmqxc.h', 'MQCHT_'],
-      ].each do |item|
+    ].each do |item|
       str << "\n# #{item[0]}\n"
       str << GenerateConst.rb_const("#{path}/#{item[1]}", item[2], item[3])
     end
@@ -126,9 +133,9 @@ END_OF_STRING
   end
 
   def self.generate(path, target_path='')
-    File.open(File.join(target_path,'constants_admin.rb'), 'w') {|file| file.write(GenerateConst.admin_consts(path))}
+    File.open(File.join(target_path, 'constants_admin.rb'), 'w') { |file| file.write(GenerateConst.admin_consts(path)) }
     puts 'Generated wmq_const_admin.rb'
-    File.open(File.join(target_path,'constants.rb'), 'w') {|file| file.write(GenerateConst.wmq_const(path))}
+    File.open(File.join(target_path, 'constants.rb'), 'w') { |file| file.write(GenerateConst.wmq_const(path)) }
     puts 'Generated wmq_const.rb'
   end
 end

data/ext/generate/generate_reason.rb

@@ -2,45 +2,45 @@ class GenerateReason
 
   # Extract Error Code Constants from Header files
   def GenerateReason.extract_const(filename, const_prefix)
-    @constants = []
+    constants = []
     File.open(filename) do |file|
       file.each do |line|
         line.rstrip!
         # Skip empty and comment lines
         if line.length > 0 && line !~ /^\s*\/\*/
           if match = /\s*#define\s+(#{const_prefix}\w+)[\(\s]+([-\dx]+)/.match(line)
-            @constants << [match[1], match[2]]
+            constants << [match[1], match[2]]
           end
         end
       end
     end
-    @constants
+    constants
   end
 
   # Extract Error Code Constants from Header files
   # Uses lazy print to collect duplicate values
   def GenerateReason.reason_case(filename, prefix)
-    last_rc = nil
+    last_rc     = nil
     last_reason = nil
-    str = ''
-    GenerateReason.extract_const(filename,prefix).each do |item|
+    str         = ''
+    GenerateReason.extract_const(filename, prefix).each do |item|
       if last_rc == item[1]
         str << "        case %-30s: return \"#{item[0]} or #{last_reason}[#{item[1]}]\";\n" % item[0]
-        last_rc = nil
+        last_rc     = nil
         last_reason = nil
       else
         str << "        case %-30s: return \"#{last_reason}[#{last_rc}]\";\n" % last_reason if last_rc
-        last_rc = item[1]
+        last_rc     = item[1]
         last_reason = item[0]
       end
     end
     str
   end
 
-# The WebSphere MQ header files have several duplicate values,
-# so need to exclude duplicates.
+  # The WebSphere MQ header files have several duplicate values,
+  # so need to exclude duplicates.
   def GenerateReason.selector_case(filename, prefix, excludes=nil, &block)
-    GenerateReason.extract_const(filename,prefix).each do |item|
+    GenerateReason.extract_const(filename, prefix).each do |item|
       next if item[0].include?('FIRST') || item[0].include?('LAST')
       next if excludes && excludes.include?(item[0])
       block.call item
@@ -65,8 +65,10 @@ char* wmq_reason(MQLONG reason_code)
     {
 END_OF_STRING
 
-    [['cmqc.h', 'MQRC_'],
-     ['cmqcfc.h', 'MQRCCF_']].each do |item|
+    [
+      ['cmqc.h', 'MQRC_'],
+      ['cmqcfc.h', 'MQRCCF_']
+    ].each do |item|
       str << GenerateReason.reason_case(path+item[0], item[1])
     end
     str << <<END_OF_STRING
@@ -77,39 +79,39 @@ END_OF_STRING
 
 END_OF_STRING
 
-    str_switch = ''
+    str_switch  = ''
     str_id_init = ''
+    values = []
 # Integer Selectors for Object Attributes
-    [['cmqc.h', 'MQIA_'],
-     ['cmqcfc.h', 'MQIACF_', ['MQIACF_ERROR_ID',
-                              'MQIACF_QUIESCE']],
-     ['cmqcfc.h', 'MQIACH_', ['MQIACH_CURRENT_SEQ_NUMBER',
-                              'MQIACH_BYTES_RCVD',
-                              'MQIACH_BUFFERS_RCVD',
-                              'MQIACH_MSGS_RCVD']],
-     ['cmqcfc.h', 'MQIAMO_'],
-     ['cmqcfc.h', 'MQIAMO64_',  ['MQIAMO64_AVG_Q_TIME',
-                                                    'MQIAMO64_Q_TIME_AVG',
-                                                    'MQIAMO64_Q_TIME_MAX',
-                                                    'MQIAMO64_Q_TIME_MIN']],
+    [
+      ['cmqc.h', 'MQIA_'],
+      ['cmqcfc.h', 'MQIACF_', ['MQIACF_ERROR_ID', 'MQIACF_QUIESCE']],
+      ['cmqcfc.h', 'MQIACH_', ['MQIACH_CURRENT_SEQ_NUMBER', 'MQIACH_BYTES_RCVD', 'MQIACH_BUFFERS_RCVD', 'MQIACH_MSGS_RCVD']],
+      ['cmqcfc.h', 'MQIAMO_'],
+      ['cmqcfc.h', 'MQIAMO64_', ['MQIAMO64_AVG_Q_TIME', 'MQIAMO64_Q_TIME_AVG', 'MQIAMO64_Q_TIME_MAX', 'MQIAMO64_Q_TIME_MIN']],
 # Integer System Selectors
-     ['cmqbc.h', 'MQIASY_'],
+      ['cmqbc.h', 'MQIASY_'],
 # Character Selectors for Object Attributes
-     ['cmqc.h', 'MQCA_',['MQCA_BASE_OBJECT_NAME']],
-     ['cmqcfc.h', 'MQCACF_'],
-     ['cmqcfc.h', 'MQCACH_'],
-     ['cmqcfc.h', 'MQCAMO_'],
+      ['cmqc.h', 'MQCA_', ['MQCA_BASE_OBJECT_NAME']],
+      ['cmqcfc.h', 'MQCACF_'],
+      ['cmqcfc.h', 'MQCACH_'],
+      ['cmqcfc.h', 'MQCAMO_'],
 # Byte String Selectors for Object Attributes
-     ['cmqc.h', 'MQBA_'],
-     ['cmqcfc.h', 'MQBACF_'],
+      ['cmqc.h', 'MQBA_'],
+      ['cmqcfc.h', 'MQBACF_'],
 # Group Selectors for Object Attributes
-     ['cmqcfc.h', 'MQGACF_'] ].each do |item|
+      ['cmqcfc.h', 'MQGACF_']
+    ].each do |item|
       str << "/* Constants #{item[1]}* from #{item[0]} */\n"
       str_switch << "        /* Constants #{item[1]}* from #{item[0]} */\n"
       str_id_init << "    /* Constants #{item[1]}* from #{item[0]} */\n"
       GenerateReason.selector_case(path+item[0], item[1], item[2]) do |const|
-        id = const[0].gsub(item[1],'').downcase
-        str_switch << "        case %-30s: return ID_#{id};\n" % const[0]
+        id = const[0].gsub(item[1], '').downcase
+        # Ignore duplicate values
+        unless values.include?(const[1])
+          str_switch << "        case %-30s: return ID_#{id};\n" % const[0]
+          values << const[1]
+        end
         str_id_init << "    ID_%-25s = rb_intern(\"#{id}\");\n" % id
         str << "static ID ID_#{id};\n"
       end
@@ -136,35 +138,31 @@ void wmq_selector(ID selector_id, PMQLONG selector_type, PMQLONG selector)
 {
     *selector_type = MQIT_INTEGER;
 END_OF_STRING
-    first = true
-    str_if = ''
+    first       = true
+    str_if      = ''
     str_id_init = ''
 # Integer Selectors for Object Attributes
-    [['cmqc.h',   'MQIA_'],
-     ['cmqcfc.h', 'MQIACF_', ['MQIACF_ERROR_ID',
-                                              'MQIACF_QUIESCE']],
-     ['cmqcfc.h', 'MQIACH_', ['MQIACH_CURRENT_SEQ_NUMBER',
-                                              'MQIACH_BYTES_RCVD',
-                                              'MQIACH_BUFFERS_RCVD']],
-     ['cmqcfc.h', 'MQIAMO_'],
-     ['cmqcfc.h', 'MQIAMO64_',  ['MQIAMO64_AVG_Q_TIME',
-                                                    'MQIAMO64_Q_TIME_AVG',
-                                                    'MQIAMO64_Q_TIME_MAX',
-                                                    'MQIAMO64_Q_TIME_MIN']],
+    [
+      ['cmqc.h', 'MQIA_'],
+      ['cmqcfc.h', 'MQIACF_', ['MQIACF_ERROR_ID', 'MQIACF_QUIESCE']],
+      ['cmqcfc.h', 'MQIACH_', ['MQIACH_CURRENT_SEQ_NUMBER', 'MQIACH_BYTES_RCVD', 'MQIACH_BUFFERS_RCVD']],
+      ['cmqcfc.h', 'MQIAMO_'],
+      ['cmqcfc.h', 'MQIAMO64_', ['MQIAMO64_AVG_Q_TIME', 'MQIAMO64_Q_TIME_AVG', 'MQIAMO64_Q_TIME_MAX', 'MQIAMO64_Q_TIME_MIN']],
 # Integer System Selectors
 #     ['cmqbc.h', 'MQIASY_'],
     ].each do |item|
-      str << "    /* Consteants #{item[1]}* from #{item[0]} */\n"
+      str << "    /* Constants #{item[1]}* from #{item[0]} */\n"
       GenerateReason.selector_case(path+item[0], item[1], item[2]) do |const|
-        str << "    if(selector_id == %-32s{ *selector = #{const[0]}; return;}\n" % "ID_#{const[0].gsub(item[1],'').downcase})"
+        str << "    if(selector_id == %-32s{ *selector = #{const[0]}; return;}\n" % "ID_#{const[0].gsub(item[1], '').downcase})"
       end
     end
     str << "\n    *selector_type = MQIT_STRING;\n\n"
 # Character Selectors for Object Attributes
-   [['cmqc.h', 'MQCA_',['MQCA_BASE_OBJECT_NAME']],
-     ['cmqcfc.h', 'MQCACF_'],
-     ['cmqcfc.h', 'MQCACH_'],
-     ['cmqcfc.h', 'MQCAMO_'],
+    [
+      ['cmqc.h', 'MQCA_', ['MQCA_BASE_OBJECT_NAME']],
+      ['cmqcfc.h', 'MQCACF_'],
+      ['cmqcfc.h', 'MQCACH_'],
+      ['cmqcfc.h', 'MQCAMO_'],
 # Byte String Selectors for Object Attributes
 #     ['cmqc.h', 'MQBA_'],
 #     ['cmqcfc.h', 'MQBACF_'],
@@ -173,7 +171,7 @@ END_OF_STRING
     ].each do |item|
       str << "    /* Constants #{item[1]}* from #{item[0]} */\n"
       GenerateReason.selector_case(path+item[0], item[1], item[2]) do |const|
-        str << "    if(selector_id == %-32s{ *selector = #{const[0]}; return;}\n" % "ID_#{const[0].gsub(item[1],'').downcase})"
+        str << "    if(selector_id == %-32s{ *selector = #{const[0]}; return;}\n" % "ID_#{const[0].gsub(item[1], '').downcase})"
       end
     end
 
@@ -183,21 +181,21 @@ END_OF_STRING
 }
 
 END_OF_STRING
-      first = true
-      str_if = ''
-      str_id_init = ''
-      GenerateReason.selector_case(path+'cmqcfc.h', 'MQCMD_') do |const|
-        if first
-          first = false
-          str_if << "    "
-        else
-          str_if << "    else "
-        end
-        id = const[0].gsub('MQCMD_','').downcase
-        str_id_init << "    ID_%-25s = rb_intern(\"#{id}\");\n" % id
-        str_if << "if(command_id == %-28s{ command = #{const[0]}; }\n" % "ID_#{const[0].gsub('MQCMD_','').downcase})"
-        str << "static ID ID_#{id};\n"
+    first       = true
+    str_if      = ''
+    str_id_init = ''
+    GenerateReason.selector_case(path+'cmqcfc.h', 'MQCMD_') do |const|
+      if first
+        first = false
+        str_if << '    '
+      else
+        str_if << '    else '
       end
+      id = const[0].gsub('MQCMD_', '').downcase
+      str_id_init << "    ID_%-25s = rb_intern(\"#{id}\");\n" % id
+      str_if << "if(command_id == %-28s{ command = #{const[0]}; }\n" % "ID_#{const[0].gsub('MQCMD_', '').downcase})"
+      str << "static ID ID_#{id};\n"
+    end
     str << <<END_OF_STRING
 
 void QueueManager_command_id_init()
@@ -217,11 +215,11 @@ MQLONG wmq_command_lookup(ID command_id)
     return command;
 }
 END_OF_STRING
-  str
+    str
   end
 
   def self.generate(path)
-    File.open('wmq_reason.c', 'w') {|file| file.write(GenerateReason.wmq_reason(path))}
+    File.open('wmq_reason.c', 'w') { |file| file.write(GenerateReason.wmq_reason(path)) }
     puts 'Generated wmq_reason.c'
   end
 end