do not export non-public types via public API

Author: thurka

visualvm/libs.profiler/lib.profiler/src/org/graalvm/visualvm/lib/jfluid/server/Monitors.java

@@ -719,11 +719,11 @@ public synchronized void next() {
 
     private static final boolean DEBUG = Boolean.getBoolean("org.graalvm.visualvm.lib.jfluid.server.Monitors");
     protected static Runtime runtime;
-    protected static SurvGenAndThreadsMonitor stMonitor;
     protected static long[] generalMNums;
     protected static long[] gcRelTime;
     protected static long[] gcStartTimes;
     protected static long[] gcFinishTimes;
+    private static SurvGenAndThreadsMonitor stMonitor;
     private static long startTimeMilis;
     private static long startTimeCounts;
     private static boolean threadsSamplingEnabled;

visualvm/libs.profiler/lib.profiler/src/org/graalvm/visualvm/lib/jfluid/server/ProfilerRuntimeCPUSampledInstr.java

@@ -56,7 +56,7 @@ void sample() {
     //~ Static fields/initializers -----------------------------------------------------------------------------------------------
 
     protected static int samplingInterval = 10;
-    protected static SamplingThread st;
+    private static SamplingThread st;
 
     //~ Methods ------------------------------------------------------------------------------------------------------------------
 

visualvm/libs.profiler/lib.profiler/src/org/graalvm/visualvm/lib/jfluid/server/ProfilerRuntimeObjLiveness.java

@@ -162,8 +162,8 @@ private void rehash() {
     //~ Static fields/initializers -----------------------------------------------------------------------------------------------
 
     protected static ReferenceQueue rq;
-    protected static WeakRefSet objSet;
-    protected static ReferenceManagerThread rmt;
+    private static WeakRefSet objSet;
+    private static ReferenceManagerThread rmt;
     protected static boolean runGCOnGetResults;
     protected static boolean objLivenessProfilingDisabled = true;
 
@@ -207,7 +207,7 @@ public static void resetProfilerCollectors() {
         // numbers were reported after resetting the results, which he (rightfully so) considered wrong
     }
 
-    public static void signalObjGC(ProfilerRuntimeObjLivenessWeakRef wr) {
+    static void signalObjGC(ProfilerRuntimeObjLivenessWeakRef wr) {
         long objectId = wr.objId;
         objSet.remove(wr);
         writeObjGCEvent(objectId);

visualvm/libs.profiler/profiler.heapwalker/src/org/graalvm/visualvm/lib/profiler/heapwalk/details/jdk/image/FieldAccessor.java

@@ -44,7 +44,7 @@ public FieldAccessor() {
         this(new InstanceBuilderRegistry());
     }
 
-    public FieldAccessor(InstanceBuilderRegistry registry) {
+    FieldAccessor(InstanceBuilderRegistry registry) {
         this.registry = registry;
     }
 
@@ -198,7 +198,7 @@ public Instance getInstance(Instance instance, String field, String clsName) thr
         return value;
     }
 
-    public <T> T build(Instance instance, String field, InstanceBuilder<T> builder, boolean allowNull)
+    <T> T build(Instance instance, String field, InstanceBuilder<T> builder, boolean allowNull)
             throws InvalidFieldException {
         return builder.convert(this, get(instance, field, Instance.class, allowNull));
     }
@@ -247,7 +247,7 @@ public <T> T[] buildArray(Instance instance, String field, Class<T> type,
      * @param allowNull never return null, throw exception instead
      * @param allowNullValues returned array can contain nulls
      */
-    public <T> T[] buildArray(Instance instance, String field, InstanceBuilder<T> builder,
+    <T> T[] buildArray(Instance instance, String field, InstanceBuilder<T> builder,
             boolean allowNull, boolean allowNullValues) throws InvalidFieldException {
         return buildArray(instance, field, builder.getType(), builder, allowNull, allowNullValues);
     }