Some style nits and add from_ical to readme

Author: seejohnrun

README.md

@@ -132,12 +132,17 @@ the schedule's start_time. Schedule start times are supported as:
 ice_cube implements its own hash-based .to_yaml, so you can quickly (and
 safely) serialize schedule objects in and out of your data store
 
+It also supports serialization to/from `ICAL`.
+
 ``` ruby
 yaml = schedule.to_yaml
 IceCube::Schedule.from_yaml(yaml)
 
 hash = schedule.to_hash
 IceCube::Schedule.from_hash(hash)
+
+ical = schedule.to_ical
+IceCube::Schedule.from_ical(ical)
 ```
 
 ---

lib/ice_cube/parsers/ical_parser.rb

@@ -40,39 +40,40 @@ def self.rule_from_ical(ical)
         when 'WKST'
           params[:wkst] = TimeUtil.ical_day_to_symbol(value)
         when 'BYSECOND'
-          params[:validations][:second_of_minute] = value.split(',').collect{ |v| v.to_i }
-        when "BYMINUTE"
-          params[:validations][:minute_of_hour] = value.split(',').collect{ |v| v.to_i }
-        when "BYHOUR"
-          params[:validations][:hour_of_day] = value.split(',').collect{ |v| v.to_i }
-        when "BYDAY"
+          params[:validations][:second_of_minute] = value.split(',').collect(&:to_i)
+        when 'BYMINUTE'
+          params[:validations][:minute_of_hour] = value.split(',').collect(&:to_i)
+        when 'BYHOUR'
+          params[:validations][:hour_of_day] = value.split(',').collect(&:to_i)
+        when 'BYDAY'
           dows = {}
           days = []
           value.split(',').each do |expr|
-              day = TimeUtil.ical_day_to_symbol(expr.strip[-2..-1])
-              if expr.strip.length > 2  # day with occurence
-                occ = expr[0..-3].to_i
-                dows[day].nil? ? dows[day] = [occ] : dows[day].push(occ)
-                days.delete(TimeUtil.sym_to_wday(day))
-              else
-                days.push TimeUtil.sym_to_wday(day) if dows[day].nil?
-              end
+            day = TimeUtil.ical_day_to_symbol(expr.strip[-2..-1])
+            if expr.strip.length > 2  # day with occurence
+              occ = expr[0..-3].to_i
+              dows[day].nil? ? dows[day] = [occ] : dows[day].push(occ)
+              days.delete(TimeUtil.sym_to_wday(day))
+            else
+              days.push TimeUtil.sym_to_wday(day) if dows[day].nil?
+            end
           end
           params[:validations][:day_of_week] = dows unless dows.empty?
           params[:validations][:day] = days unless days.empty?
-        when "BYMONTHDAY"
-          params[:validations][:day_of_month] = value.split(',').collect{ |v| v.to_i }
-        when "BYMONTH"
-          params[:validations][:month_of_year] = value.split(',').collect{ |v| v.to_i }
-        when "BYYEARDAY"
-          params[:validations][:day_of_year] = value.split(',').collect{ |v| v.to_i }
-        when "BYSETPOS"
+        when 'BYMONTHDAY'
+          params[:validations][:day_of_month] = value.split(',').collect(&:to_i)
+        when 'BYMONTH'
+          params[:validations][:month_of_year] = value.split(',').collect(&:to_i)
+        when 'BYYEARDAY'
+          params[:validations][:day_of_year] = value.split(',').collect(&:to_i)
+        when 'BYSETPOS'
         else
-          raise "Invalid or unsupported rrule command : #{name}"
+          raise "Invalid or unsupported rrule command: #{name}"
         end
       end
 
       params[:interval] ||= 1
+
       # WKST only valid for weekly rules
       params.delete(:wkst) unless params[:freq] == 'weekly'