best to still have ruby 1.8 (syntax) compatibility ... as we're building the jar on 1.6.8

Author: kares

lib/arjdbc/mysql/bulk_change_table.rb

@@ -9,9 +9,7 @@ module BulkChangeTable
       ChangeColumnDefinition = ActiveRecord::ConnectionAdapters::ChangeColumnDefinition if AR41
 
       # @override
-      def supports_bulk_alter?
-        true
-      end
+      def supports_bulk_alter?; true end
 
       def bulk_change_table(table_name, operations)
         sqls = operations.map do |command, args|
@@ -94,7 +92,7 @@ def rename_column_sql(table_name, column_name, new_column_name)
       end unless AR41
 
       def rename_column_sql(table_name, column_name, new_column_name)
-        options = { name: new_column_name }
+        options = { :name => new_column_name }
 
         if column = columns(table_name).find { |c| c.name == column_name.to_s }
           options[:default] = column.default

test/db/mysql/types_test.rb

@@ -10,9 +10,9 @@ def test_binary_types
   end
 
   def test_error_handling
-    assert_raise ActiveRecord::ActiveRecordError { type_to_sql(:binary, 5_000_000_000) }
-    assert_raise ActiveRecord::ActiveRecordError { type_to_sql(:integer, 10) }
-    assert_raise ActiveRecord::ActiveRecordError { type_to_sql(:text, 5_000_000_000) }
+    assert_raise(ActiveRecord::ActiveRecordError) { type_to_sql(:binary, 5_000_000_000) }
+    assert_raise(ActiveRecord::ActiveRecordError) { type_to_sql(:integer, 10) }
+    assert_raise(ActiveRecord::ActiveRecordError) { type_to_sql(:text, 5_000_000_000) }
   end
 
   def type_to_sql(*args)