Arianne RPG / Bugs / #5382 [Stendhal] Check performance impact of ModifierHandler

#5382 [Stendhal] Check performance impact of ModifierHandler

Status: closed-fixed

Owner: nobody

Labels: None

Priority: 7

Updated: 2012-02-03

Created: 2012-01-06

Creator: Hendrik Brummermann

Private: No

GuidedEntityModifierHandler might have a negative impact on memory because it is added to every creature.

It might negatively impact CPU because getBaseSpeed() is not inlinable by Hotspot anymore.

The changed might be perfectly fine, but need verification.

Discussion

Hendrik Brummermann - 2012-01-06

priority: 5 --> 7
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Hendrik Brummermann - 2012-01-06

summary: [Stendhal] Check performance impact of GuidedEntityModifierH --> [Stendhal] Check performance impact of ModifierHandler
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Hendrik Brummermann - 2012-01-06

The same is true for RPEntityModifierHandler.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Markus - 2012-02-03

status: open --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Markus - 2012-02-03

Due to lazy initialization of the ModifierHandler this should be fixed now.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous