justinmclean commented on issue #75: Set all padding values at once URL: https://github.com/apache/royale-asjs/pull/75#issuecomment-344096760 Also re PAYG: With left , right, top and bottom padding version the example program release size is 83,026 bytes With single padding version the example program release size is 82,960 bytes. So using the single padding attribute will save you a few bytes - 66 in this case. Not significant but it doesn't add to the size so there's no PAYG cost there. Again if the padding attribute is not used then the function is optimised out / removed so there's no PAYG size cost if you don't use the single padding attribute. Just out of interest I looked at a mixed version: 1 single padding and 1 set of padding attributes 83,082 bytes. 2 sets of padding attributes 83,148 bytes. So even if you mix them there is a few bytes saved and no PAYG cost. Runtime wise the left right, top and bottom version it needs to make four function calls to set the padding. The single padding version will be faster as it only calls one function. Not that you could probably measure it in any meaningful way or that it would have a noticeable impact on a simple application but it may provide a small benefit in complex layout, item renderers or the like. There is no runtime PAYG cost if you use or don't use the single padding attribute. So can you please explain how you find this not PAYG?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
