Project

General

Profile

Actions

Bug #4748

closed

documentation updates: Curses and friends are 100% documented

Added by vbatts (Vincent Batts) over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
ruby 1.9.3dev (2011-05-20 trunk 31647) [x86_64-linux]
Backport:
[ruby-core:36355]

Description

well 100% per the rdoc report.
Sadly there are a host of constants that are not being picked up by RDoc (I intend to follow up on that)


Files

Actions #1

Updated by drbrain (Eric Hodel) over 11 years ago

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

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


  • ext/curses/curses.c: Complete documentation. Patch by Vincent
    Batts. [Ruby 1.9 - Bug #4748]

Updated by mame (Yusuke Endoh) over 11 years ago

  • Status changed from Closed to Open

Hello,

    • === Constants
    • For whatever reason, these are not picked up in the RDoc
    • These are Integer of the colors.
      • COLOR_BLACK = 0
      • COLOR_RED = 1
      • COLOR_GREEN = 2
      • COLOR_YELLOW = 3
      • COLOR_BLUE = 4
      • COLOR_MAGENTA = 5
      • COLOR_CYAN = 6
      • COLOR_WHITE = 7

These actual values must not be documented.

These constants just export ones of curses.h.
Some environments have a different integer set. For example,
I heard that COLOR_RED is 4 in windows.
In addition, Ruby curses library can change the actual value in
future. For example, it might be better for such a constant to
have a Symbol.

  • /*
  • * Document-const: Curses::COLOR_BLACK
    
  • *
    
  • * Integer of the color black
    
  • */
    

For the same reason, "Integer" should not be mentioned.

--
Yusuke Endoh

Actions #3

Updated by drbrain (Eric Hodel) over 11 years ago

  • Status changed from Open to Closed

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


  • ext/curses/curses.c: Remove color constants block.
    [Ruby 1.9 - Bug #4748]

Updated by vbatts (Vincent Batts) over 11 years ago

Err. adding an updated patch, that removes the TODO regarding the constants

Updated by vbatts (Vincent Batts) over 11 years ago

oh for the love. wrong issue updated :(
intended for issue #4880

Updated by vbatts (Vincent Batts) over 11 years ago

oh for the love. wrong issue updated :(
intended for issue #4880

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0