[ruby-core:63816] [ruby-trunk - Feature #10058] [PATCH] Fix some coding styles

From: ngotogenome@...
Date: 2014-07-18 09:06:52 UTC
List: ruby-core #63816
Issue #10058 has been updated by Naohisa Goto.


> There are both send :xxx and send(:xxx) expressions. So, I Unify with send(:xxx)

I think "send" should be replaced with "__send__", because the Japanese version of reference manual for Object#send says that libraries should use "__send__".

http://docs.ruby-lang.org/ja/2.1.0/class/Object.html#I___SEND__

"send" は "__send__" にするのがよいと思います。なぜなら、Object#send の日本語のリファレンスマニュアルには「"send" が再定義された場合に備えて別名 "__send__" も 用意されており、ライブラリではこちらを使うべきです。」と書いてあるからです。


----------------------------------------
Feature #10058: [PATCH] Fix some coding styles
https://bugs.ruby-lang.org/issues/10058#change-47853

* Author: gogo tanaka
* Status: Open
* Priority: Low
* Assignee: 
* Category: 
* Target version: 
----------------------------------------
When I read codes, I notice some issues about coding style.

If you don't mind that, you can ignore my patches. It's just coding style problem.

コードを読んでいる中でコーディングスタイルに関して気になる事があったのでPATCHを投げますが、

こちら(redmine, rubyコミュニティ)のコーディングスタイルに対する温度感が掴めてないので

気にする事でなければ、無視してください. 

あげると切りがないので今回は特に気になる以下の三点に絞りました. 

* There are both `&&` and `and`. So, I Unify with `&&` 

* There are both `send :xxx` and `send(:xxx)`  expressions. So, I Unify with `send(:xxx)` 

*  I reorder methods. I think Matrix#real should be upon Matrix#imaginary(From what I see complex.c)

---Files--------------------------------
unify_product_operator.patch (879 Bytes)
reorder_Matrix#real_and_Matrix_imaginary.patch (1.3 KB)
use_method_with_parenthesis.patch (1021 Bytes)


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

In This Thread

Prev Next