Summary: | SemanticChangedCommand should not be here | ||
---|---|---|---|
Product: | [Modeling] Sirius | Reporter: | Esteban DUGUEPEROUX <esteban.dugueperoux> |
Component: | Diagram | Assignee: | Project Inbox <sirius.diagram-inbox> |
Status: | NEW --- | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | florian.barbin, pierre-charles.david |
Version: | 2.0.0 | Keywords: | triaged |
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux | ||
Whiteboard: |
Description
Esteban DUGUEPEROUX
2014-11-28 11:39:04 EST
Can you provide more details about the consequences (with a scenario)? You describe a solution but this issue will be complicated to test during the validation without more details. Esteban, from your description this looks like a performance issue (causing too many unneeded DRefreshable.refresh() and EditPart.refresh() calls). Is this correct or can this also produce invalid results (and not just slowdowns)? If this is about performance, please add the corresponding keyword, and if possible some measures (ideally times, otherwise suspiciously high invocation counts for some methods). Note that this may be one of the causes for what is described in bug #443518. This bugzilla is to show that some refresh is called in bad places. Fixing that will improve performance, I will try to do some performance measures. |