Branch: refs/heads/master
Home: https://github.com/jenkinsci/analysis-core-plugin
Commit: 243c4a493546f0acf50decf066f92d61f6168ced
https://github.com/jenkinsci/analysis-core-plugin/commit/243c4a493546f0acf50decf066f92d61f6168ced
Author: Kohsuke Kawaguchi <[email protected]>
Date: 2012-06-20 (Wed, 20 Jun 2012)
Changed paths:
M src/main/java/hudson/plugins/analysis/core/AbstractAnnotationParser.java
M src/main/java/hudson/plugins/analysis/util/TreeStringBuilder.java
M src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
Log Message:
-----------
Apply the memory footprint improvement when parsing reports from tools.
The series of changes leading up to this change so far only affects when
loading FileAnnotation[] via XStream, so it won't help when we are
parsing output from tools.
This change plugs that last hole, by applying data reuse at the end
of AbstractAnnotationParser.parse().
I think reasonable people can argue about who's implementation details
this processing is. I'm not particularly attached to this
implementation, so any adjustments are welcome as long as it achieves
the goal of having this dedup process kick in for newly parsed reports.
For example, doing it here means it doesn't kick in when someone calls
parse(InputStream), but this has an advantages of not needing to
update every subtypes, and letting the caller of AnnotationParser does
this has its own set of downsides.
Commit: ead06a96623170f74bdea6ed4b37a8ef5c567077
https://github.com/jenkinsci/analysis-core-plugin/commit/ead06a96623170f74bdea6ed4b37a8ef5c567077
Author: Ulli Hafner <[email protected]>
Date: 2012-06-20 (Wed, 20 Jun 2012)
Changed paths:
M src/main/java/hudson/plugins/analysis/core/AbstractAnnotationParser.java
M src/main/java/hudson/plugins/analysis/util/TreeStringBuilder.java
M src/main/java/hudson/plugins/analysis/util/model/AbstractAnnotation.java
Log Message:
-----------
Merge pull request #11 from kohsuke/master
Apply the memory footprint improvement when parsing reports from tools.
Compare:
https://github.com/jenkinsci/analysis-core-plugin/compare/40d560ee81e3...ead06a966231
