clarified different requirements to a mount point

Author: overheadhunter

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

@@ -11,9 +11,40 @@ public enum MountFeature {
 	 * and the builder requires {@link MountBuilder#setMountFlags(String)}.
 	 */
 	MOUNT_FLAGS,
-	MOUNT_POINT_EMPTY_DIR,
-	MOUNT_POINT_DRIVE_LETTER,
-	MOUNT_POINT_PATH_PREFIX,
+
+	/**
+	 * With the exception of a provider-supplied default mount point, the mount point must be an existing dir.
+	 * <p>
+	 * This option is mutually exclusive with {@link #MOUNT_WITHIN_EXISTING_PARENT}.
+	 *
+	 * @see #DEFAULT_MOUNT_POINT
+	 */
+	MOUNT_TO_EXISTING_DIR,
+
+	/**
+	 * With the exception of a provider-supplied default mount point, the mount point must be a non-existing
+	 * child within an existing parent.
+	 * <p>
+	 * This option is mutually exclusive with {@link #MOUNT_TO_EXISTING_DIR}.
+	 *
+	 * @see #DEFAULT_MOUNT_POINT
+	 */
+	MOUNT_WITHIN_EXISTING_PARENT,
+
+	/**
+	 * The mount point may be a drive letter.
+	 *
+	 * @see #MOUNT_TO_EXISTING_DIR
+	 * @see #MOUNT_WITHIN_EXISTING_PARENT
+	 */
+	MOUNT_AS_DRIVE_LETTER,
+
+	/**
+	 * The provider supports suggesting a default mount point via {@link MountProvider#getDefaultMountPoint(String)}.
+	 * <p>
+	 * The default mount point is guaranteed to be supported by the mount builder, regardless of its normal restrictions.
+	 */
+	DEFAULT_MOUNT_POINT,
 
 	/**
 	 * The builder supports {@link MountBuilder#setReadOnly(boolean)}
@@ -24,16 +55,15 @@ public enum MountFeature {
 	 * The mount supports {@link Mount#unmountForced()}.
 	 */
 	UNMOUNT_FORCED,
-	ON_EXIT_ACTION,
 
 	/**
-	 * The provider supports {@link MountProvider#getDefaultPort()}
-	 * and the builder requires {@link MountBuilder#setPort(int)}.
+	 * The mount supports triggering an event handler, which is somehow triggered at a specific time. // TODO what?
 	 */
-	PORT,
+	ON_EXIT_ACTION,
 
 	/**
-	 * The provider supports {@link MountProvider#getDefaultMountPoint(String)}
+	 * The provider supports {@link MountProvider#getDefaultPort()}
+	 * and the builder requires {@link MountBuilder#setPort(int)}.
 	 */
-	DEFAULT_MOUNT_POINT
+	PORT
 }

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

@@ -60,7 +60,8 @@ static Stream<MountProvider> get() {
 	 * A suitable mount point suggested by this provider.
 	 * <p>
 	 * Other than caller-provided mount points, the mount point suggested by this method can be
-	 * passed to {@link MountBuilder#setMountpoint(Path)} without further ado.
+	 * passed to {@link MountBuilder#setMountpoint(Path)} and is guaranteed to fulfill the builder's requirements
+	 * without further ado.
 	 *
 	 * @param mountPointSuffix String used in the generation of a mount point.
 	 * @return A path to a possible mount point.