[Jruby-extras-devel] [765] trunk/activerecord-jdbc/lib/jdbc_adapter: Remove some unnecessary warnings

[nobody]

Title: [765] trunk/activerecord-jdbc/lib/jdbc_adapter: Remove some unnecessary warnings

Revision

765

Author

olabini

Date

2007-10-11 07:27:00 -0400 (Thu, 11 Oct 2007)

Log Message

Remove some unnecessary warnings

Modified Paths

• trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_db2.rb
• trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_derby.rb
• trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_hsqldb.rb
• trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_postgre.rb

Diff

Modified: trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_db2.rb (764 => 765)

--- trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_db2.rb	2007-10-10 13:20:19 UTC (rev 764)
+++ trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_db2.rb	2007-10-11 11:27:00 UTC (rev 765)
@@ -35,7 +35,7 @@
       def cast_to_date_or_time(value)
         return value if value.is_a? Date
         return nil if value.blank?
-        guess_date_or_time (value.is_a? Time) ? value : cast_to_time(value)
+        guess_date_or_time((value.is_a? Time) ? value : cast_to_time(value))
       end
 
       def cast_to_time(value)

Modified: trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_derby.rb (764 => 765)

--- trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_derby.rb	2007-10-10 13:20:19 UTC (rev 764)
+++ trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_derby.rb	2007-10-11 11:27:00 UTC (rev 765)
@@ -55,7 +55,7 @@
       def cast_to_date_or_time(value)
         return value if value.is_a? Date
         return nil if value.blank?
-        guess_date_or_time (value.is_a? Time) ? value : cast_to_time(value)
+        guess_date_or_time((value.is_a? Time) ? value : cast_to_time(value))
       end
 
       def cast_to_time(value)

Modified: trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_hsqldb.rb (764 => 765)

--- trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_hsqldb.rb	2007-10-10 13:20:19 UTC (rev 764)
+++ trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_hsqldb.rb	2007-10-11 11:27:00 UTC (rev 765)
@@ -40,7 +40,7 @@
       def cast_to_date_or_time(value)
         return value if value.is_a? Date
         return nil if value.blank?
-        guess_date_or_time (value.is_a? Time) ? value : cast_to_time(value)
+        guess_date_or_time((value.is_a? Time) ? value : cast_to_time(value))
       end
 
       def cast_to_time(value)

Modified: trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_postgre.rb (764 => 765)

--- trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_postgre.rb	2007-10-10 13:20:19 UTC (rev 764)
+++ trunk/activerecord-jdbc/lib/jdbc_adapter/jdbc_postgre.rb	2007-10-11 11:27:00 UTC (rev 765)
@@ -47,7 +47,7 @@
       def cast_to_date_or_time(value)
         return value if value.is_a? Date
         return nil if value.blank?
-        guess_date_or_time (value.is_a? Time) ? value : cast_to_time(value)
+        guess_date_or_time((value.is_a? Time) ? value : cast_to_time(value))
       end
 
       def cast_to_time(value)
@@ -240,7 +240,7 @@
       # construct a clean list of column names from the ORDER BY clause, removing
       # any asc/desc modifiers
       order_columns = order_by.split(',').collect { |s| s.split.first }
-      order_columns.delete_if &:blank?
+      order_columns.delete_if(&:blank?)
       order_columns = order_columns.zip((0...order_columns.size).to_a).map { |s,i| "#{s} AS alias_#{i}" }
 
       # return a DISTINCT ON() clause that's distinct on the columns we want but includes

_______________________________________________
Jruby-extras-devel mailing list
Jruby-extras-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/jruby-extras-devel