From: qlli.illb@... Date: 2014-07-19T19:20:43+00:00 Subject: [ruby-core:63883] [ruby-trunk - Feature #10058] [PATCH] Fix some coding styles Issue #10058 has been updated by gogo tanaka. Marc-Andre Lafortune wrote: > Yes, ideally. If they can be discussed separately, they should be separate issues. > > No need for this issue, it's already been committed. I see, I try this next time. Thank you for your committing my patches. ---------------------------------------- Feature #10058: [PATCH] Fix some coding styles https://bugs.ruby-lang.org/issues/10058#change-47909 * Author: gogo tanaka * Status: Open * Priority: Low * Assignee: Marc-Andre Lafortune * 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/