Merge branch 'master' into pr/31

Author: Col-E

core/src/main/java/module-info.java

@@ -1,12 +1,20 @@
-module bento.fx {
+import org.jspecify.annotations.NullMarked;
+
+/**
+ * A docking system for JavaFX.
+ * <p>
+ * Everything is open/exported. Do whatever you want.
+ * </p>
+ */
+@NullMarked
+open module bento.fx {
 	requires static org.jspecify;
 
 	requires javafx.base;
 	requires javafx.graphics;
 	requires javafx.controls;
 	requires java.desktop;
 
-	// Just open/export everything. Do whatever you want.
 	exports software.coley.bentofx;
 	exports software.coley.bentofx.building;
 	exports software.coley.bentofx.control;
@@ -18,15 +26,4 @@
 	exports software.coley.bentofx.path;
 	exports software.coley.bentofx.search;
 	exports software.coley.bentofx.util;
-	opens software.coley.bentofx;
-	opens software.coley.bentofx.building;
-	opens software.coley.bentofx.control;
-	opens software.coley.bentofx.control.canvas;
-	opens software.coley.bentofx.dockable;
-	opens software.coley.bentofx.event;
-	opens software.coley.bentofx.layout;
-	opens software.coley.bentofx.layout.container;
-	opens software.coley.bentofx.path;
-	opens software.coley.bentofx.search;
-	opens software.coley.bentofx.util;
-}
+}

core/src/main/java/software/coley/bentofx/Bento.java

@@ -2,7 +2,6 @@
 
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
-import org.jspecify.annotations.NonNull;
 import software.coley.bentofx.building.ControlsBuilding;
 import software.coley.bentofx.building.DockBuilding;
 import software.coley.bentofx.building.PlaceholderBuilding;
@@ -34,106 +33,90 @@ public class Bento {
 	private final DockableDragDropBehavior dragDropBehavior = newDragDropBehavior();
 	private final DockableClickBehavior clickBehavior = newClickBehavior();
 
-	@NonNull
 	protected EventBus newEventBus() {
 		return new EventBus();
 	}
 
-	@NonNull
 	protected SearchHandler newSearchHandler() {
 		return new SearchHandler(this);
 	}
 
-	@NonNull
 	protected StageBuilding newStageBuilding() {
 		return new StageBuilding(this);
 	}
 
-	@NonNull
 	protected ControlsBuilding newControlsBuilding() {
 		return new ControlsBuilding();
 	}
 
-	@NonNull
 	protected DockBuilding newDockBuilding() {
 		return new DockBuilding(this);
 	}
 
-	@NonNull
 	protected PlaceholderBuilding newPlaceholderBuilding() {
 		return new PlaceholderBuilding();
 	}
 
-	@NonNull
 	protected DockableDragDropBehavior newDragDropBehavior() {
 		return new DockableDragDropBehavior() {};
 	}
 
-	@NonNull
 	protected DockableClickBehavior newClickBehavior() {
 		return new DockableClickBehavior() {};
 	}
 
 	/**
 	 * @return Bus for handling event firing and event listeners.
 	 */
-	@NonNull
 	public EventBus events() {
 		return eventBus;
 	}
 
 	/**
 	 * @return Search operations.
 	 */
-	@NonNull
 	public SearchHandler search() {
 		return searchHandler;
 	}
 
 	/**
 	 * @return Builders for {@link DragDropStage}.
 	 */
-	@NonNull
 	public StageBuilding stageBuilding() {
 		return stageBuilding;
 	}
 
 	/**
 	 * @return Builders for various bento UI controls.
 	 */
-	@NonNull
 	public ControlsBuilding controlsBuilding() {
 		return controlsBuilding;
 	}
 
 	/**
 	 * @return Builders for {@link DockContainer} and {@link Dockable}.
 	 */
-	@NonNull
 	public DockBuilding dockBuilding() {
 		return dockBuilding;
 	}
 
 	/**
 	 * @return Builders for placeholder content.
 	 */
-	@NonNull
 	public PlaceholderBuilding placeholderBuilding() {
 		return placeholderBuilding;
 	}
 
 	/**
 	 * @return Behavior implementation for drag-drop operations.
 	 */
-	@NonNull
 	public DockableDragDropBehavior getDragDropBehavior() {
 		return dragDropBehavior;
 	}
 
 	/**
 	 * @return Behavior implementation for click operations.
 	 */
-	@NonNull
 	public DockableClickBehavior getClickBehavior() {
 		return clickBehavior;
 	}
@@ -144,7 +127,6 @@ public DockableClickBehavior getClickBehavior() {
 	 * @see #registerRoot(DockContainerRootBranch)
 	 * @see #unregisterRoot(DockContainerRootBranch)
 	 */
-	@NonNull
 	public ObservableList<DockContainerRootBranch> getRootContainers() {
 		return rootContainersView;
 	}
@@ -155,7 +137,7 @@ public ObservableList<DockContainerRootBranch> getRootContainers() {
 	 *
 	 * @return {@code true} when registered.
 	 */
-	public boolean registerRoot(@NonNull DockContainerRootBranch container) {
+	public boolean registerRoot(DockContainerRootBranch container) {
 		if (!rootContainers.contains(container)) {
 			rootContainers.add(container);
 			eventBus.fire(new DockEvent.RootContainerAdded(container));
@@ -170,7 +152,7 @@ public boolean registerRoot(@NonNull DockContainerRootBranch container) {
 	 *
 	 * @return {@code true} when unregistered.
 	 */
-	public boolean unregisterRoot(@NonNull DockContainerRootBranch container) {
+	public boolean unregisterRoot(DockContainerRootBranch container) {
 		if (rootContainers.remove(container)) {
 			eventBus.fire(new DockEvent.RootContainerRemoved(container));
 			return true;

core/src/main/java/software/coley/bentofx/BentoBacked.java

@@ -1,7 +1,5 @@
 package software.coley.bentofx;
 
-import org.jspecify.annotations.NonNull;
-
 /**
  * Outline of an object with access to its originating {@link Bento} instance.
  *
@@ -11,6 +9,5 @@ public interface BentoBacked {
 	/**
 	 * @return Bento instance responsible for this object.
 	 */
-	@NonNull
 	Bento getBento();
-}
+}

core/src/main/java/software/coley/bentofx/Identifiable.java

@@ -1,7 +1,5 @@
 package software.coley.bentofx;
 
-import org.jspecify.annotations.NonNull;
-
 /**
  * Outline of an <i>(ideally uniquely)</i> identifiable object.
  *
@@ -11,7 +9,6 @@ public interface Identifiable {
 	/**
 	 * @return This objects identifier.
 	 */
-	@NonNull
 	String getIdentifier();
 
 	/**
@@ -20,5 +17,5 @@ public interface Identifiable {
 	 *
 	 * @return {@code true} when the other object has the same identifier.
 	 */
-	boolean matchesIdentity(@NonNull Identifiable other);
-}
+	boolean matchesIdentity(Identifiable other);
+}

core/src/main/java/software/coley/bentofx/building/CanvasFactory.java

@@ -1,6 +1,5 @@
 package software.coley.bentofx.building;
 
-import org.jspecify.annotations.NonNull;
 import software.coley.bentofx.control.canvas.PixelCanvas;
 import software.coley.bentofx.layout.container.DockContainerLeaf;
 
@@ -16,6 +15,5 @@ public interface CanvasFactory {
 	 *
 	 * @return New canvas.
 	 */
-	@NonNull
-	PixelCanvas newCanvas(@NonNull DockContainerLeaf container);
+	PixelCanvas newCanvas(DockContainerLeaf container);
 }

core/src/main/java/software/coley/bentofx/building/ContentWrapperFactory.java

@@ -1,6 +1,5 @@
 package software.coley.bentofx.building;
 
-import org.jspecify.annotations.NonNull;
 import software.coley.bentofx.control.ContentWrapper;
 import software.coley.bentofx.control.HeaderPane;
 import software.coley.bentofx.layout.container.DockContainerLeaf;
@@ -17,6 +16,5 @@ public interface ContentWrapperFactory {
 	 *
 	 * @return Newly created content wrapper.
 	 */
-	@NonNull
-	ContentWrapper newContentWrapper(@NonNull DockContainerLeaf container);
+	ContentWrapper newContentWrapper(DockContainerLeaf container);
 }

core/src/main/java/software/coley/bentofx/building/ControlsBuilding.java

@@ -2,7 +2,6 @@
 
 import javafx.geometry.Orientation;
 import javafx.geometry.Side;
-import org.jspecify.annotations.NonNull;
 import org.jspecify.annotations.Nullable;
 import software.coley.bentofx.control.ContentWrapper;
 import software.coley.bentofx.control.Header;
@@ -32,7 +31,6 @@ public class ControlsBuilding implements HeaderPaneFactory, HeadersFactory, Head
 	/**
 	 * @return Factory for creating {@link HeaderPane}.
 	 */
-	@NonNull
 	public HeaderPaneFactory getHeaderPaneFactory() {
 		return headerPaneFactory;
 	}
@@ -51,7 +49,6 @@ public void setHeaderPaneFactory(@Nullable HeaderPaneFactory headerPaneFactory)
 	/**
 	 * @return Factory for creating {@link Headers}.
 	 */
-	@NonNull
 	public HeadersFactory getHeadersFactory() {
 		return headersFactory;
 	}
@@ -70,7 +67,6 @@ public void setHeadersFactory(@Nullable HeadersFactory headersFactory) {
 	/**
 	 * @return Factory for creating {@link Header}.
 	 */
-	@NonNull
 	public HeaderFactory getHeaderFactory() {
 		return headerFactory;
 	}
@@ -89,7 +85,6 @@ public void setHeaderFactory(@Nullable HeaderFactory headerFactory) {
 	/**
 	 * @return Factory for creating {@link ContentWrapper}.
 	 */
-	@NonNull
 	public ContentWrapperFactory getContentWrapperFactory() {
 		return contentWrapperFactory;
 	}
@@ -108,7 +103,6 @@ public void setContentWrapperFactory(@Nullable ContentWrapperFactory contentWrap
 	/**
 	 * @return Factory for creating {@link PixelCanvas}.
 	 */
-	@NonNull
 	public CanvasFactory getCanvasFactory() {
 		return canvasFactory;
 	}
@@ -124,33 +118,28 @@ public void setCanvasFactory(@Nullable CanvasFactory canvasFactory) {
 		this.canvasFactory = canvasFactory;
 	}
 
-	@NonNull
 	@Override
-	public HeaderPane newHeaderPane(@NonNull DockContainerLeaf container) {
+	public HeaderPane newHeaderPane(DockContainerLeaf container) {
 		return headerPaneFactory.newHeaderPane(container);
 	}
 
-	@NonNull
 	@Override
-	public Headers newHeaders(@NonNull DockContainerLeaf container, @NonNull Orientation orientation, @NonNull Side side) {
+	public Headers newHeaders(DockContainerLeaf container, Orientation orientation, Side side) {
 		return headersFactory.newHeaders(container, orientation, side);
 	}
 
-	@NonNull
 	@Override
-	public Header newHeader(@NonNull Dockable dockable, @NonNull HeaderPane parentPane) {
+	public Header newHeader(Dockable dockable, HeaderPane parentPane) {
 		return headerFactory.newHeader(dockable, parentPane);
 	}
 
-	@NonNull
 	@Override
-	public ContentWrapper newContentWrapper(@NonNull DockContainerLeaf container) {
+	public ContentWrapper newContentWrapper(DockContainerLeaf container) {
 		return contentWrapperFactory.newContentWrapper(container);
 	}
 
-	@NonNull
 	@Override
-	public PixelCanvas newCanvas(@NonNull DockContainerLeaf container) {
+	public PixelCanvas newCanvas(DockContainerLeaf container) {
 		return canvasFactory.newCanvas(container);
 	}
 }