Project

General

Profile

Actions

Feature #5088

closed

Refactor and Document vm_method.c / method.h

Added by lazaridis.com (Lazaridis Ilias) over 12 years ago. Updated almost 12 years ago.

Status:
Closed
Target version:
-
[ruby-core:38444]

Description

=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

Updated by gnufied (hemant kumar) over 12 years ago

@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.

Updated by lazaridis.com (Lazaridis Ilias) over 12 years ago

=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

Updated by lazaridis.com (Lazaridis Ilias) over 12 years ago

=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

Updated by mame (Yusuke Endoh) about 12 years ago

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

Updated by ko1 (Koichi Sasada) almost 12 years ago

  • 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.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0