From: ngotogenome@...
Date: 2014-07-18T09:40:00+00:00
Subject: [ruby-core:63819] [ruby-trunk - Feature #10058] [PATCH] Fix some	coding styles

Issue #10058 has been updated by Naohisa Goto.


> There are both && and and. So, I Unify with &&

Be careful that "&&" and "and" have different operator precedence, and they can not always be simply replaceable.

 && ��� and ���������������������������������������������������������������������������������������������������������������

http://qiita.com/fukayatsu/items/920ad8e4d099c3fec890


By the way, FYI, Rails coding guideline says "Prefer &&/|| over and/or".

������������������������������������������Rails���������������������������&&/||������������and/or������������������������������������������������������������

http://guides.rubyonrails.org/contributing_to_ruby_on_rails.html#follow-the-coding-conventions



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

* 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/