Skip to content

Commit 9e668cd

Browse files
Andreas Sewemebigfatguy
Andreas Sewe
authored andcommitted
Remove the YourKit profiling callback (spotbugs#289)
1 parent 197014e commit 9e668cd

File tree

5 files changed

+3
-116
lines changed

5 files changed

+3
-116
lines changed

CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ This is the changelog for SpotBugs. This follows [Keep a Changelog v0.3](http://
44

55
## Unreleased (2017/??/??)
66

7+
### Removed
8+
9+
* The `YourKitProfiler` class has been removed and the `findbugs.yourkit.enabled` system property is no longer supported ([#289](https://github.com/spotbugs/spotbugs/issues/289))
710

811
## 3.1.0-RC4 (2017/Jul/21)
912

-103 KB
Binary file not shown.

spotbugs/src/doc/index.html

-7
Original file line numberDiff line numberDiff line change
@@ -344,13 +344,6 @@ <h2>
344344
project, including founding work by <a href="http://goose.ycp.edu/~dhovemey/">David Hovemeyer</a>
345345
and the web cloud infrastructure by Keith Lea.
346346
</p>
347-
<p>
348-
YourKit is kindly supporting open source projects with its full-featured Java Profiler. YourKit, LLC
349-
is creator of innovative and intelligent tools for profiling Java and .NET applications. Take a look
350-
at YourKit's leading software products: <a href="http://www.yourkit.com/java/profiler/index.jsp">YourKit
351-
Java Profiler</a> and <a href="http://www.yourkit.com/.net/profiler/index.jsp">YourKit .NET
352-
Profiler</a>.
353-
</p>
354347
<p>
355348
The FindBugs project also uses <a href="http://www.atlassian.com/software/fisheye/">FishEye</a> and
356349
<a href="http://www.atlassian.com/software/clover/">Clover</a>, which are generously provided by <a

spotbugs/src/main/java/edu/umd/cs/findbugs/FindBugs2.java

-11
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,6 @@
6969
import edu.umd.cs.findbugs.detect.NoteSuppressedWarnings;
7070
import edu.umd.cs.findbugs.filter.FilterException;
7171
import edu.umd.cs.findbugs.log.Profiler;
72-
import edu.umd.cs.findbugs.log.YourKitController;
7372
import edu.umd.cs.findbugs.plan.AnalysisPass;
7473
import edu.umd.cs.findbugs.plan.ExecutionPlan;
7574
import edu.umd.cs.findbugs.plan.OrderingConstraintException;
@@ -118,8 +117,6 @@ public class FindBugs2 implements IFindBugsEngine {
118117

119118
private ExecutionPlan executionPlan;
120119

121-
private final YourKitController yourkitController = new YourKitController();
122-
123120
private String currentClassName;
124121

125122
private FindBugsProgress progress;
@@ -986,7 +983,6 @@ private void analyzeApplication() throws InterruptedException {
986983
bugReporter.getProjectStats().setReferencedClasses(referencedClassSet.size());
987984
for (Iterator<AnalysisPass> passIterator = executionPlan.passIterator(); passIterator.hasNext();) {
988985
AnalysisPass pass = passIterator.next();
989-
yourkitController.advanceGeneration("Pass " + passCount);
990986
// The first pass is generally a non-reporting pass which
991987
// gathers information about referenced classes.
992988
boolean isNonReportingFirstPass = multiplePasses && passCount == 0;
@@ -1047,10 +1043,6 @@ public Collection<ClassDescriptor> getOutEdges(ClassDescriptor e) {
10471043
classCollection.size(), classDescriptor);
10481044
}
10491045
count++;
1050-
if (!isNonReportingFirstPass && count % 1000 == 0) {
1051-
yourkitController.advanceGeneration(String.format("Pass %d.%02d", passCount, count/1000));
1052-
}
1053-
10541046

10551047
// Check to see if class is excluded by the class screener.
10561048
// In general, we do not want to screen classes from the
@@ -1121,9 +1113,6 @@ public Collection<ClassDescriptor> getOutEdges(ClassDescriptor e) {
11211113
}
11221114
}
11231115

1124-
if (!passIterator.hasNext()) {
1125-
yourkitController.captureMemorySnapshot();
1126-
}
11271116
// Call finishPass on each detector
11281117
for (Detector2 detector : detectorList) {
11291118
detector.finishPass();

spotbugs/src/main/java/edu/umd/cs/findbugs/log/YourKitController.java

-98
This file was deleted.

0 commit comments

Comments
 (0)