Hi, I’ve noticed in a few places there are some unnecessary trace statements in the SDK. Like for instance here [1].
I’m thinking we probably shouldn’t have trace statements in production code. Sonar cube flags it as an issue. [2] Anyone think otherwise? On the JS side these are only omitted when goog.DEBUG is defined but still be a little expensive at it calls slice before doing that. Thanks, Justin 1. https://github.com/apache/flex-asjs/blob/fd709d137e5f740fd3a7f7dfbcde0e898e25f103/frameworks/projects/DragDrop/src/main/flex/org/apache/flex/html/beads/controllers/DragMouseController.as 2. https://builds.apache.org/analysis/component_issues/index?id=org.apache.flex.flexjs.framework%3Aflexjs-framework-parent#resolved=false|severities=CRITICAL|rules=flex%3AS1951