Project

General

Profile

Actions
Copy link

Bug #8745

closed

Add documentation for RSS::Utils

Bug #8745: Add documentation for RSS::Utils

Added by steveklabnik (Steve Klabnik) about 12 years ago. Updated about 12 years ago.

Status:
Closed
Assignee:
zzak (zzak _)
Target version:
-
ruby -v:
2.1.0-dev
Backport:
1.9.3: UNKNOWN, 2.0.0: UNKNOWN
[ruby-core:56414]

Description

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

Files

Download all files
0001-Add-documentation-for-RSS-Utils.patch (3.3 KB) 0001-Add-documentation-for-RSS-Utils.patch steveklabnik (Steve Klabnik), 08/07/2013 03:31 AM
0001-Add-documentation-for-RSS-Utils.patch (3.52 KB) 0001-Add-documentation-for-RSS-Utils.patch steveklabnik (Steve Klabnik), 08/07/2013 03:56 AM

Updated by steveklabnik (Steve Klabnik) about 12 years ago Actions
Copy link
 #1 [ruby-core:56415]

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

Updated by zzak (zzak _) about 12 years ago Actions
Copy link
 #2 [ruby-core:56416]

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

Updated by zzak (zzak _) about 12 years ago Actions
Copy link
 #3 [ruby-core:56417]

  • Status changed from Open to Assigned

Updated by steveklabnik (Steve Klabnik) about 12 years ago Actions
Copy link
 #4 [ruby-core:56418]

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.

Updated by zzak (zzak _) about 12 years ago Actions
Copy link
 #5

  • 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]
Actions
Copy link

Also available in: PDF Atom