[#83107] Alias Enumerable#include? to Enumerable#includes? — Alberto Almagro <albertoalmagro@...>

Hello,

9 messages 2017/10/04

[ruby-core:83441] [Ruby trunk Bug#13741][Assigned] A documentation bug of IO#putc

From: naruse@...
Date: 2017-10-21 10:29:49 UTC
List: ruby-core #83441
Issue #13741 has been updated by naruse (Yui NARUSE).

Status changed from Open to Assigned
Assignee changed from naruse (Yui NARUSE) to shugo (Shugo Maeda)

OK, could you commit it?

----------------------------------------
Bug #13741: A documentation bug of IO#putc
https://bugs.ruby-lang.org/issues/13741#change-67412

* Author: shugo (Shugo Maeda)
* Status: Assigned
* Priority: Normal
* Assignee: shugo (Shugo Maeda)
* Target version: 
* ruby -v: 
* Backport: 2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN
----------------------------------------
The documentation of IO#putc says:


```
 *  call-seq:
 *     ios.putc(obj)    -> obj
 *
 *  If <i>obj</i> is <code>Numeric</code>, write the character whose code is
 *  the least-significant byte of <i>obj</i>, otherwise write the first byte
 *  of the string representation of <i>obj</i> to <em>ios</em>. Note: This
 *  method is not safe for use with multi-byte characters as it will truncate
 *  them.
```

However, it supports multi-byte characters in the string argument form
since r29447:

```
$ ruby -e 'putc "あいうえお"'
あ
```

How about to change the documentation?

```
 *  If <i>obj</i> is <code>Numeric</code>, write the character whose code is
 *  the least-significant byte of <i>obj</i>.
 *  If <i>obj</i> is <code>String</code>, write the first character
 *  of <i>obj</i> to <em>ios</em>.
 *  Otherwise, raise <code>TypeError</code>.
```

It may be better to change the behavior of the numeric argument form
to support multi-byte characters if <em>ios</em> has encoding
information, but it's a different issue.




-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>

In This Thread

Prev Next