Project

General

Profile

Bug #8745

Add documentation for RSS::Utils

Added by Steve Klabnik almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
ruby -v:
2.1.0-dev
[ruby-core:56414]

Description

This adds both top-level as well as method-level documentation for RSS::Utils.

0001-Add-documentation-for-RSS-Utils.patch View (3.3 KB) Steve Klabnik, 08/07/2013 03:31 AM

0001-Add-documentation-for-RSS-Utils.patch View (3.52 KB) Steve Klabnik, 08/07/2013 03:56 AM

Associated revisions

Revision 42415
Added by Zachary Scott almost 3 years ago

  • lib/rss/utils.rb: [DOC] RSS::Utils by Steve Klabnik [Bug #8745]

Revision 42415
Added by Zachary Scott almost 3 years ago

  • lib/rss/utils.rb: [DOC] RSS::Utils by Steve Klabnik [Bug #8745]

Revision 42415
Added by Zachary Scott almost 3 years ago

  • lib/rss/utils.rb: [DOC] RSS::Utils by Steve Klabnik [Bug #8745]

History

#1 [ruby-core:56415] Updated by Steve Klabnik almost 3 years ago

I have attached a better patch that explains an edge case of an extra method, as well as fixes indentation for code samples.

#2 [ruby-core:56416] Updated by Zachary Scott almost 3 years ago

I like the non-instance_eval example better, it is more clear

#3 [ruby-core:56417] Updated by Zachary Scott almost 3 years ago

  • Status changed from Open to Assigned

#4 [ruby-core:56418] Updated by Steve Klabnik almost 3 years ago

Sure thing, whatever you think is best :)

I think that since 0 is a special case, we can get away with one level of indirection.

#5 Updated by Zachary Scott almost 3 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r42415.
Steve, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/rss/utils.rb: [DOC] RSS::Utils by Steve Klabnik [Bug #8745]

Also available in: Atom PDF