More lean API. Removed features might be readded later:

* removed OnExitAction feature
* removed reveal() funtion
* renamed getAccessPoint()

Author: infeo

src/main/java/org/cryptomator/integrations/mount/Mount.java

@@ -1,7 +1,6 @@
 package org.cryptomator.integrations.mount;
 
 import java.nio.file.Path;
-import java.util.function.Consumer;
 
 /**
  * Handle to control the lifecycle of a mounted file system.
@@ -10,12 +9,12 @@
  */
 public interface Mount extends AutoCloseable {
 
-	//TODO: either this or reveal method, not both
-	Path getAccessPoint();
-
-	//TODO: is this needed? why not just let the consumer reveal?
-	// See WebDAV: LinuxGioMounter and LinuxGvfsMounter -> no path, just a command
-	void reveal(Consumer<Path> cmd);
+	/**
+	 * Returns the absolute OS path, where this mount can be accessed.
+	 *
+	 * @return Absolute path to the mountpoint.
+	 */
+	Path getMountpoint();
 
 	/**
 	 * Unmounts the mounted Volume.

src/main/java/org/cryptomator/integrations/mount/MountBuilder.java

@@ -4,7 +4,6 @@
 import org.jetbrains.annotations.Range;
 
 import java.nio.file.Path;
-import java.util.function.Consumer;
 
 /**
  * Builder to mount a filesystem.
@@ -14,8 +13,6 @@
  */
 public interface MountBuilder {
 
-	//TODO: Idea: every setter verifies the set and can throw an IllegalArgumentException
-
 	/**
 	 * Sets the mount point.
 	 *
@@ -26,12 +23,6 @@ public interface MountBuilder {
 	@Contract("_ -> this")
 	MountBuilder setMountpoint(Path mountPoint);
 
-	// TODO: in what legacy mounter impl is this used?
-	@Contract("_ -> this")
-	default MountBuilder setOnExitAction(Consumer<Throwable> onExitAction) {
-		throw new UnsupportedOperationException();
-	}
-
 	/**
 	 * Sets mount flags.
 	 *

src/main/java/org/cryptomator/integrations/mount/MountFeature.java

@@ -56,11 +56,6 @@ public enum MountFeature {
 	 */
 	UNMOUNT_FORCED,
 
-	/**
-	 * The mount supports triggering an event handler, which is somehow triggered at a specific time. // TODO what?
-	 */
-	ON_EXIT_ACTION,
-
 	/**
 	 * The provider supports {@link MountProvider#getDefaultPort()}
 	 * and the builder requires {@link MountBuilder#setPort(int)}.

src/main/java/org/cryptomator/integrations/mount/MountProvider.java

@@ -9,22 +9,6 @@
 import java.util.Set;
 import java.util.stream.Stream;
 
-// Vorteile:
-//  * IoC: Ein Provider weiß was er kann und was nicht!
-//  * Wegfall der komplizierten Mountpoint Logik
-//  * App beschränkt sich mehr auf GUI und Logik darum
-//  * Einfaches ändern/entfernen eines Providers
-//  * Mounten ist eine Systemintegration!
-//  * Tests!?
-//  * Illegal Mount Options Check?
-//  * In der GUI sieht der Nutzer sofort, was sich ändert.
-// Nachteile:
-//  * Code-Duplikation
-//  * WebDAV-ServerPort? (-> Legacy Option in Preferences?)
-// Ungeklärt:
-//  * Anzeigen von Fehler dem User (-> Translations nutzbar?)
-//  * Langfristiger Wechsel zu eigenständigen Prozessen?
-
 /**
  * A mechanism to mount a file system.
  *