Bug #4711

documentation updates: Net::POP3 and Net::Telnet are 100%

Added by Vincent Batts almost 3 years ago. Updated almost 3 years ago.

[ruby-core:36227]
Status:Closed
Priority:Normal
Assignee:Eric Hodel
Category:doc
Target version:1.9.3
ruby -v:ruby 1.9.3dev (2011-05-16 trunk 31589) [x86_64-linux] Backport:

Description

per http://blog.segment7.net/2011/05/09/ruby-1-9-3-documentation-challenge

This patch includes the following bits.

class Net::Telnet # is documented

# in file lib/net/telnet.rb
def close(); end

end

class Net::POP3 # is documented

# in file lib/net/pop.rb
Revision = nil
# in file lib/net/pop.rb
def certs(); end

# in file lib/net/pop.rb
def createsslparams(verifyorparams = {}, certs = nil); end

# in file lib/net/pop.rb
def ssl_params(); end

# in file lib/net/pop.rb
def use_ssl?(); end

# in file lib/net/pop.rb
def verify(); end

# in file lib/net/pop.rb
def command(); end

# in file lib/net/pop.rb
def disable_ssl(); end

# in file lib/net/pop.rb
def do_finish(); end

# in file lib/net/pop.rb
def do_start(account, password); end

# in file lib/net/pop.rb
def logging(msg); end

# in file lib/net/pop.rb
def on_connect(); end

end

ruby_doc_updates-20110516-1013_vbatts.patch Magnifier (2.59 KB) Vincent Batts, 05/16/2011 11:15 PM

ruby_doc_updates-20110516-1013_vbatts.patch Magnifier (2.74 KB) Vincent Batts, 05/17/2011 05:22 AM

Associated revisions

Revision 31602
Added by Eric Hodel almost 3 years ago

  • lib/net/pop.rb: Improve documentation. Patch by Vincent Batts. [Ruby 1.9 - Bug #4711]
    • lib/net/telnet.rb: ditto

Revision 31728
Added by Eric Hodel almost 3 years ago

  • lib/net/pop.rb: Hide implementation details from RDoc. [Ruby 1.9 - Bug #4711]

History

#1 Updated by Vincent Batts almost 3 years ago

meh, i missed a single line comment for def disable_ssl(); end

see the attached updated patch

#2 Updated by Eric Hodel almost 3 years ago

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

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


  • lib/net/pop.rb: Improve documentation. Patch by Vincent Batts. [Ruby 1.9 - Bug #4711]
    • lib/net/telnet.rb: ditto

#3 Updated by Yusuke Endoh almost 3 years ago

  • Status changed from Closed to Open

Hello,

  • # internal method for Net::POP3.start
    def do_start(account, password)

  • # Does nothing
    def onconnect
    end
    private :on
    connect

  • # nil's out the:

  • # - mails

  • # - number counter for mails

  • # - number counter for bytes

  • # - quits the current command, if any
    def do_finish

Is any comment needed there?
If so, they should be more verbose.
I think that :nodoc: is a good idea here.

Yusuke Endoh mame@tsg.ne.jp

#4 Updated by Eric Hodel almost 3 years ago

  • Status changed from Open to Closed

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


  • lib/net/pop.rb: Hide implementation details from RDoc. [Ruby 1.9 - Bug #4711]

#5 Updated by Eric Hodel almost 3 years ago

What is the onconnect method for? Net::HTTP has an empty onconnect method also, is it for subclasses of Net::HTTP and Net::POP3?

#6 Updated by Usaku NAKAMURA almost 3 years ago

What is the onconnect method for? Net::HTTP has an empty onconnect method also, is it for subclasses of Net::HTTP and Net::POP3?

I guess so.

Also available in: Atom PDF