From e650d4104d7e7c1b6f3530889a7cdc56857eb19b Mon Sep 17 00:00:00 2001
From: Ben Fry <fry@processing.org>
Date: Fri, 3 Feb 2017 11:58:22 -0500
Subject: [PATCH] deal with a handful of warnings

---
 java/src/processing/mode/java/Debugger.java           |  2 +-
 .../mode/java/pdex/PreprocessingService.java          | 11 +++++------
 java/src/processing/mode/java/runner/Runner.java      |  2 +-
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/java/src/processing/mode/java/Debugger.java b/java/src/processing/mode/java/Debugger.java
index c4be8c3a7..5d7eecb57 100644
--- a/java/src/processing/mode/java/Debugger.java
+++ b/java/src/processing/mode/java/Debugger.java
@@ -1407,7 +1407,7 @@ public class Debugger {
   }
 
 
-  private void logitse(Throwable t) {
+  static private void logitse(Throwable t) {
     Messages.loge("incompatible thread state?", t);
   }
 
diff --git a/java/src/processing/mode/java/pdex/PreprocessingService.java b/java/src/processing/mode/java/pdex/PreprocessingService.java
index ce535a92d..0827e3ffe 100644
--- a/java/src/processing/mode/java/pdex/PreprocessingService.java
+++ b/java/src/processing/mode/java/pdex/PreprocessingService.java
@@ -72,7 +72,6 @@ import processing.mode.java.preproc.PdePreprocessor.Mode;
 /**
  * The main error checking service
  */
-@SuppressWarnings("unchecked")
 public class PreprocessingService {
 
   protected final JavaEditor editor;
@@ -93,7 +92,7 @@ public class PreprocessingService {
   private CompletableFuture<PreprocessedSketch> preprocessingTask = new CompletableFuture<>();
 
   private CompletableFuture<?> lastCallback =
-      new CompletableFuture() {{
+      new CompletableFuture<Object>() {{
         complete(null); // initialization block
       }};
 
@@ -568,8 +567,8 @@ public class PreprocessingService {
   }
 
 
-  private List<String> buildSketchLibraryClassPath(JavaMode mode,
-                                                   List<ImportStatement> programImports) {
+  static private List<String> buildSketchLibraryClassPath(JavaMode mode,
+                                                          List<ImportStatement> programImports) {
     StringBuilder classPath = new StringBuilder();
 
     programImports.stream()
@@ -590,7 +589,7 @@ public class PreprocessingService {
   }
 
 
-  private List<String> buildJavaRuntimeClassPath() {
+  static private List<String> buildJavaRuntimeClassPath() {
     StringBuilder classPath = new StringBuilder();
 
     // Java runtime
@@ -654,7 +653,7 @@ public class PreprocessingService {
   /**
    * Ignore processing packages, java.*.*. etc.
    */
-  private boolean ignorableImport(String packageName) {
+  static private boolean ignorableImport(String packageName) {
     return (packageName.startsWith("java.") ||
             packageName.startsWith("javax."));
   }
diff --git a/java/src/processing/mode/java/runner/Runner.java b/java/src/processing/mode/java/runner/Runner.java
index c383fc2c8..1223ee752 100644
--- a/java/src/processing/mode/java/runner/Runner.java
+++ b/java/src/processing/mode/java/runner/Runner.java
@@ -127,7 +127,7 @@ public class Runner implements MessageConsumer {
    * Has the user screwed up their hosts file?
    * https://github.com/processing/processing/issues/4738
    */
-  private void checkLocalHost() throws SketchException {
+  static private void checkLocalHost() throws SketchException {
     try {
       InetAddress address = InetAddress.getByName("localhost");
       if (!address.getHostAddress().equals("127.0.0.1")) {