Comply with checkstyle

Author: SamCarlberg

core/src/main/java/edu/wpi/grip/core/Main.java

@@ -70,9 +70,6 @@ public void start(String[] args) throws IOException, InterruptedException {
       project.open(new File(projectPath));
     }
 
-    // Open a project from a .grip file specified on the command line
-    project.open(new File(projectPath));
-
     pipelineRunner.startAsync();
 
     // This is done in order to indicate to the user using the deployment UI that this is running
@@ -99,4 +96,4 @@ public final void onExceptionClearedEvent(ExceptionClearedEvent event) {
         + "Event");
   }
 
-}
+}

core/src/main/java/edu/wpi/grip/core/events/RunStartedEvent.java

@@ -1,8 +1,8 @@
 package edu.wpi.grip.core.events;
 
 /**
- * An event fired when the pipeline starts running. This is guaranteed to be followed by a corresponding
- * {@link RunStoppedEvent}.
+ * An event fired when the pipeline starts running. This is guaranteed to be followed by a
+ * corresponding {@link RunStoppedEvent}.
  */
 public class RunStartedEvent {
 }

core/src/main/java/edu/wpi/grip/core/events/RunStoppedEvent.java

@@ -1,8 +1,11 @@
 package edu.wpi.grip.core.events;
 
 /**
- * An event fired when the pipeline stops running. This is guaranteed to follow a corresponding {@link RunStartedEvent}.
- * This is different from {@link RenderEvent} in that it will <i>always</i> be fired when the pipeline runs.
+ * An event fired when the pipeline stops running. This is guaranteed to follow a corresponding
+ * {@link RunStartedEvent}.
+ *
+ * <p>This is different from {@link RenderEvent} in that it will <i>always</i> be fired when the
+ * pipeline runs.
  */
 public class RunStoppedEvent {
 }

core/src/main/java/edu/wpi/grip/core/exception/GripException.java

@@ -8,12 +8,12 @@
  */
 public abstract class GripException extends RuntimeException {
 
-    public GripException(String message) {
-        super(message);
-    }
+  public GripException(String message) {
+    super(message);
+  }
 
-    public GripException(String message, Throwable cause) {
-        super(message, cause);
-    }
+  public GripException(String message, Throwable cause) {
+    super(message, cause);
+  }
 
 }

core/src/main/java/edu/wpi/grip/core/exception/GripServerException.java

@@ -1,15 +1,16 @@
 package edu.wpi.grip.core.exception;
 
 /**
- * An exception thrown when something goes wrong in the {@link edu.wpi.grip.core.http.GripServer GripServer}.
+ * An exception thrown when something goes wrong in the
+ * {@link edu.wpi.grip.core.http.GripServer GripServer}.
  */
 public class GripServerException extends GripException {
 
-    public GripServerException(String message) {
-        super(message);
-    }
+  public GripServerException(String message) {
+    super(message);
+  }
 
-    public GripServerException(String message, Throwable cause) {
-        super(message, cause);
-    }
+  public GripServerException(String message, Throwable cause) {
+    super(message, cause);
+  }
 }

core/src/main/java/edu/wpi/grip/core/http/ContextStore.java

@@ -2,10 +2,12 @@
 
 import com.google.inject.Singleton;
 
-import javax.annotation.Nullable;
 import java.util.HashSet;
 import java.util.Set;
 
+import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
+
 import static com.google.common.base.Preconditions.checkNotNull;
 
 /**
@@ -14,42 +16,43 @@
 @Singleton
 public class ContextStore {
 
-    private final Set<String> store = new HashSet<>();
-
-    /**
-     * Records the given context.
-     *
-     * @param context the context to record. This cannot be null.
-     * @throws IllegalArgumentException if the given context has already been claimed
-     */
-    public void record(String context) throws IllegalArgumentException {
-        checkNotNull(context);
-        if (!store.add(context)) {
-            throw new IllegalArgumentException("Context is already claimed: " + context);
-        }
-    }
-
-    /**
-     * Erases the given context from this store, if it's present. If {@code context} is {@code null}, this will
-     * do nothing and return {@code false}.
-     *
-     * @param context the context to erase
-     * @return true if the context was erased, false if it wasn't erased or if it wasn't present to begin with.
-     */
-    public boolean erase(@Nullable String context) {
-        return store.remove(context);
-    }
-
-    /**
-     * Checks if the given context has been recorded in this store. If {@code context} is {@code null}, this will
-     * return {@code false}.
-     *
-     * @param context the context to check
-     * @return true if the given context has been recorded in this store
-     */
-    public boolean contains(@Nullable String context) {
-        return store.contains(context);
+  private final Set<String> store = new HashSet<>();
+
+  /**
+   * Records the given context.
+   *
+   * @param context the context to record. This cannot be null.
+   * @throws IllegalArgumentException if the given context has already been claimed
+   */
+  public void record(@Nonnull String context) throws IllegalArgumentException {
+    checkNotNull(context);
+    if (!store.add(context)) {
+      throw new IllegalArgumentException("Context is already claimed: " + context);
     }
+  }
+
+  /**
+   * Erases the given context from this store, if it's present. If {@code context} is {@code null},
+   * this will do nothing and return {@code false}.
+   *
+   * @param context the context to erase
+   * @return true if the context was erased, false if it wasn't erased
+   *         or if it wasn't present to begin with.
+   */
+  public boolean erase(@Nullable String context) {
+    return store.remove(context);
+  }
+
+  /**
+   * Checks if the given context has been recorded in this store.
+   * If {@code context} is {@code null}, this will return {@code false}.
+   *
+   * @param context the context to check
+   * @return true if the given context has been recorded in this store
+   */
+  public boolean contains(@Nullable String context) {
+    return store.contains(context);
+  }
 
 
 }