flex-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maurice Amsellem (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLEX-33865) ConstraintLayout / LayoutElementHelper are memory inefficient (and slow)
Date Sat, 02 Nov 2013 20:55:18 GMT

    [ https://issues.apache.org/jira/browse/FLEX-33865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13812165#comment-13812165
] 

Maurice Amsellem commented on FLEX-33865:
-----------------------------------------

Hi Benoit,

the first optimization (_constraintColumns instead of constraintColumns) is fine. It avoids
an uncessary slice().

I have a few questions regarding the second optimization ( passing an Array to avoid creation
of an Array):
1)  you say the method is slow.  Is it faster with your optimization? Can you provider before/after
timing
2) There are other places where the method is called without the second parameter (eg. parseConstraintValue
above).  In this case, the second unused argument and the addtional tests may actually *slow
down* the call.
3) Can you provide a test application that shows where your optimization is effective



> ConstraintLayout / LayoutElementHelper are memory inefficient (and slow)
> ------------------------------------------------------------------------
>
>                 Key: FLEX-33865
>                 URL: https://issues.apache.org/jira/browse/FLEX-33865
>             Project: Apache Flex
>          Issue Type: Improvement
>          Components: Mobile: Performance, Spark: Layout
>    Affects Versions: Apache Flex 4.11.0
>         Environment: mobile desktop
>            Reporter: Benoit Wiart
>            Assignee: Maurice Amsellem
>              Labels: mobile, performance
>         Attachments: 0001-ConstraintLayout-optimizations.patch, layout-1-desktop-memory.png,
layout-2-desktop-memory.png
>
>
> ConstraintLayout / LayoutElementHelper are doing too many memory allocation.
> it's really bad on mobile
> the attached screenshots were taken on desktop



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message