Project

General

Profile

Actions

Patch #26130

closed

Refactor "multiple_values_detail" struct creation

Added by Thomas Löber almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Hi,

In the IssuesHelper#details_to_strings method (source:trunk/app/helpers/issues_helper.rb@16387#L373) a new class (Struct) is created each time the method is called:

  multiple_values_detail = Struct.new(:property, :prop_key, :custom_field, :old_value, :value)

Although classes are garbage collected in recent Ruby versions, it might be better to use a constant definition outside the method, like:

  MultipleValuesDetail = Struct.new(:property, :prop_key, :custom_field, :old_value, :value)

Regards,
Thomas

Actions #1

Updated by Go MAEDA almost 7 years ago

  • Category changed from Performance to Code cleanup/refactoring
Actions #2

Updated by Go MAEDA almost 7 years ago

  • Description updated (diff)
Actions #3

Updated by Jean-Philippe Lang almost 7 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Target version set to 4.0.0

Change committed, thanks.

Actions #4

Updated by Toshi MARUYAMA almost 7 years ago

  • Description updated (diff)
Actions #5

Updated by Toshi MARUYAMA almost 7 years ago

  • Target version changed from 4.0.0 to 4.1.0

I think target version should be version:3.5.0 because there is no 3.5-stable in Subversion.

Actions #6

Updated by Jean-Philippe Lang over 6 years ago

  • Target version changed from 4.1.0 to 4.0.0
Actions

Also available in: Atom PDF