Feature #5088: Refactor and Document vm_method.c / method.h

• Status: Closed
• Assignee: ko1 (Koichi Sasada)
• Target version: -

[lazaridis.com (Lazaridis Ilias)]

=begin
(not a "Feature" but a "Task")

Refactoring and documentation of the following files:

• source:vm_method.c
• source:method.h
• other indirectly affected files

This will not alter the functionality in any way, but will change only whitespace, structure and naming.

The goals are

• to make the method handling easier to understand.
• to enable more developers to maintain the code
• To simplify processing/implementation of issues: #4893, #5005
• To simplify the completition/extension of the method-handling
• to prepare for a later redesign of vm_method.c (towards "methods are objects")

(patches follow by time, if no objecting is given)

=end

[gnufied (hemant kumar)]

@Ilias,

Please submit any improvement with a patch. It is pointless to open a request without a patch and with due respect - wastes Ruby-Core valuable time.

[lazaridis.com (Lazaridis Ilias)]

=begin
hemant kumar wrote:

@Ilias,

Please submit any improvement with a patch. It is pointless to open a request without a patch

Mr. Kumar, I don't know who you are, but the rule you state is not existent within this project.

and with due respect - wastes Ruby-Core valuable time.

I guess it distracts the list from the relevant stuff, like making jokes: #5054 ?

As I've possibly not expressed myself clear:

• Task announced, asking for general objections.
• If no objections, I'll continue and provide the patches (which can of course be objected/rejected)

=end

[lazaridis.com (Lazaridis Ilias)]

=begin

You can follow the rework on this location:

((URL:https://github.com/lazaridis-com/ruby/tree/refactor_vm_method))

What is the preferred method? Sending refactoring patches one-by-one for commit, or as a one big patch when it's finished?

=end

[mame (Yusuke Endoh)]

• Status changed from Open to Assigned
• Assignee set to ko1 (Koichi Sasada)

[ko1 (Koichi Sasada)]

• Status changed from Assigned to Closed

I think we need to discuss about each spec, before implementation (of course, a proposal with an implementation is welcome). Please persuade matz before you try to commit your patch.