[JIRA] [cppcheck] (JENKINS-19437) Implement load on demand functionality in Cppcheck

Sat, 08 Mar 2014 04:47:33 -0800

Code changed in jenkins
User: Michal Turek
Path:
src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckResult.java
src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckSourceContainer.java
src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/index.jelly
src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckResult/summary.jelly
http://jenkins-ci.org/commit/cppcheck-plugin/5db2c791e4ceb93f5cafce525ddbde5047aa0b97
Log:
JENKINS-17363 Ludicrously slow load time [with lazyloading]
JENKINS-19437 Implement load on demand functionality in Cppcheck

  • Lazy loaded details are stored in CppcheckResult member variable and never released by Java garbace collector since a reference to them exists forever. All build.xml files are loaded during Jenkins startup and never released, CppcheckResult objects are part of them. This is kind of a memory leak. The update will cause slight lower performance but releasing of the memory is much more important for long run tasks/daemons/servers.
  • Method lazyLoadSourceContainer() updated to return the data instead of their caching in a member variable.
  • If cached CppcheckSourceContainer object can't be loaded, an empty one will be created. Different constructor used to remove unnecessary exception catches. Recently added related if removed from CppcheckSourceContainer.
  • Condition moved from index.jelly to summary.jelly to be able to reuse the data and remove one unnecessary loading (performance).
  • Construction "key = +key" has no meaning, updated to "+key" to resolve a warning.
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira

--
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to