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

Reply via email to