Radio Buttons

Created RadioButton and RadioManager.
They work together well I guess.
😄

docs/CONTRIBUTING.md

@@ -1,69 +0,0 @@
-# Contributing Guidelines
-
-This document lists conventions adopted by Progressia developers.
-
-## git
-
-### Branches
-Progressia repository contains a `master` branch and several "feature" branches.
-
-`master` is expected to contain a version of the game suitable for demonstration and forking/branching. Do not commit directly to `master` without OLEGSHA's approval.
-- `master` must always correctly build without compiler warnings (see below).
-- `master` must always pass all unit tests.
-- `master` must always be able to launch successfully.
-- `master` must always only contain working features.
-- `master` should not contain excessive debug code.
-- `master` must always have its code and filenames formatted (see below).
-
-"Feature" branches are branches dedicated to the development of a single feature. When the feature reaches completion the branch is merged into `master` and removed. Intermediate merges into `master` may occur when some fitting milestone is reached. Intermediate merges from `master` may be done as necessary. Merges between "feature" branches should generally be avoided.
-
-When beginning work on a new feature, create a new branch based on `master` (or on another "feature" branch if absolutely necessary). Use `all-small-with-dashes` to name the branch: `add-trees` or `rebalance-plastics` are good names. Do not fix unrelated bugs or work on unrelated features in a "feature" branch - create a new one, switch to an existing one or commit directly to `master` if the changes are small enough.
-
-"Feature" branches may not be formatted properly. Formatting is required before merging into `master` or other branches.
-
-### Commits
-- Commits must leave the branch in a state that builds without compiler warnings (see below).
-- Changes should be grouped in commits semantically. Avoid committing many small related changes in sequence; if necessary, wait and accumulate them. Avoid committing unrelated changes together; if necessary, split staged changes into several commits. This should normally result in about 1-2 commits for a day's work.
-- Commit bulk changes (renaming, formatting, ...) separately. Don't ever commit whitespace changes outside formatting commits.
-- Message format:
-
-```
-Short description of changes
-<empty line>
-- Enumeration of changes
-  - Nest when appropriate
-- Use dashes only
-- List not needed for small commits
-```
-
-Example:
-
-```
-Changed packages for relations, renamed Face to ShapePart
-
-- Added BlockRelation as an abstract superclass to existing relations
-  - Must be given an absolute "up" direction before use
-- Moved AbsFace, AbsRelation and BlockRelation to .world.rels
-- Renamed Face to ShapePart to reduce confusion with AbsFace
-```
-
-- Only commit changes described in the commit message. Please double-check staged files before committing.
-- Avoid merge conflicts. Pull before committing.
-- Better sign commits than not. See: [git](https://git-scm.com/book/en/v2/Git-Tools-Signing-Your-Work), [GitHub](https://docs.github.com/en/github/authenticating-to-github/managing-commit-signature-verification).
-
-## Code
-
-### Warnings
-Make sure that all committed code contains no compiler warnings. This specifically includes unused imports, unused private members, missing `@Override`s and warnings related to generics.
-
-Warnings about unknown tokens in `@SuppressWarnings` are temporarily ignored. Please disable them in your IDE.
-
-### Code Style
-Formatting code is important.
-
-- The format is specified within the files inside `/templates_and_presets/eclipse_ide`. Import the specifications into Eclipse or IntelliJ IDEA and use the IDEs' format feature. Alternatively format the code manually in accordance with existing files.
-- Only use tabs for indentation. Never indent with spaces even when wrapping lines. This is to ensure that indentation does not break when tab width is different.
-- Don't use `I` prefix for interfaces (not `IDoable` but `Doable`).
-- Prioritize readability over compactness. Do not hesitate to use (very) long identifiers if they aid comprehension.
-- Document all mathematics unless it is trivial, especially when using math notation for variable names.
-- Use proper English when writing comments. Avoid boxes in comments. Use `//` for single-line comments.

src/main/java/ru/windcorp/jputil/ArrayUtil.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil;
 
 import java.lang.reflect.Array;
@@ -611,7 +611,8 @@ public class ArrayUtil {
 		int end = offset + length;
 		if (end > arrayLength || offset < 0)
 			throw new IllegalArgumentException(
-					"Array contains [0; " + arrayLength + "), requested [" + offset + "; " + end + ")");
+				"Array contains [0; " + arrayLength + "), requested [" + offset + "; " + end + ")"
+			);
 
 		return length;
 	}
@@ -627,7 +628,8 @@ public class ArrayUtil {
 
 		if (end > arrayLength || start < 0)
 			throw new IllegalArgumentException(
-					"Array contains [0; " + arrayLength + "), requested [" + start + "; " + end + ")");
+				"Array contains [0; " + arrayLength + "), requested [" + start + "; " + end + ")"
+			);
 
 		return end;
 	}

src/main/java/ru/windcorp/jputil/CSVWriter.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil;
 
 import java.io.OutputStream;

src/main/java/ru/windcorp/jputil/PrimitiveUtil.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil;
 
 import java.util.HashMap;
@@ -30,8 +30,18 @@ public class PrimitiveUtil {
 	private static final Map<Class<?>, Object> PRIMITIVE_TO_NULL = new HashMap<>();
 
 	static {
-		for (Class<?> boxed : new Class<?>[] { Boolean.class, Byte.class, Short.class, Character.class, Integer.class,
+		for (
-				Long.class, Float.class, Double.class }) {
+			Class<?> boxed : new Class<?>[] {
+				Boolean.class,
+				Byte.class,
+				Short.class,
+				Character.class,
+				Integer.class,
+				Long.class,
+				Float.class,
+				Double.class
+			}
+		) {
 			try {
 				PRIMITIVE_TO_BOXED.put((Class<?>) boxed.getField("TYPE").get(null), boxed);
 			} catch (Exception e) {

src/main/java/ru/windcorp/jputil/SyncStreams.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil;
 
 import java.util.function.*;
@@ -40,7 +40,8 @@ import java.util.stream.Stream;
 
 /**
  * Contains static methods to create {@link Stream Streams} that synchronize
- * their <a href=
+ * their
+ * <a href=
  * "https://docs.oracle.com/javase/8/docs/api/java/util/stream/package-summary.html#StreamOps">
  * terminal operations</a> on a given monitor.
  * 
@@ -49,7 +50,7 @@ import java.util.stream.Stream;
  */
 
 // SonarLint: "Stream.peek" should be used with caution (java:S3864)
-// We are implementing Stream, so peek() is required.
+//   We are implementing Stream, so peek() is required.
 @SuppressWarnings("squid:S3864")
 
 public class SyncStreams {
@@ -1069,18 +1070,21 @@ public class SyncStreams {
 	}
 
 	/**
-	 * Wraps the given {@link Stream} to make all <a href=
+	 * Wraps the given {@link Stream} to make all
+	 * <a href=
 	 * "https://docs.oracle.com/javase/8/docs/api/java/util/stream/package-summary.html#StreamOps">
 	 * terminal operations</a> acquire the provided monitor's lock before
-	 * execution. Intermediate operations return streams that are also
+	 * execution. Intermediate operations
-	 * synchronized on the same object. The created stream will behave
+	 * return streams that are also synchronized on the same object. The created
-	 * identically to the provided stream in all other aspects. Use this to
+	 * stream will behave identically
-	 * synchronize access to stream's source.
+	 * to the provided stream in all other aspects. Use this to synchronize
+	 * access to stream's source.
 	 * <p>
 	 * <i>The returned {@code Stream}'s {@link Stream#iterator() iterator()} and
-	 * {@link Stream#spliterator() spliterator()} methods return regular
+	 * {@link Stream#spliterator()
-	 * non-synchronized iterators and spliterators respectively</i>. It is the
+	 * spliterator()} methods return regular non-synchronized iterators and
-	 * user's responsibility to avoid concurrency issues:
+	 * spliterators respectively</i>. It
+	 * is the user's responsibility to avoid concurrency issues:
 	 * 
 	 * <pre>
 	 * synchronized (stream.getMonitor()) {
@@ -1099,17 +1103,14 @@ public class SyncStreams {
 	 * stream.forEach(System.out::println); // Should never throw a ConcurrentModificationException
 	 * </pre>
 	 * 
-	 * @param <T>
+	 * @param <T>     the class of objects in the Stream
-	 *            the class of objects in the Stream
+	 * @param stream  the stream to wrap.
-	 * @param stream
+	 * @param monitor the object that the stream will use for synchronization.
-	 *            the stream to wrap.
+	 *                When {@code null}, the stream
-	 * @param monitor
+	 *                will synchronize on itself.
-	 *            the object that the stream will use for synchronization. When
-	 *            {@code null}, the stream will synchronize on itself.
 	 * @return a {@link SyncStream SyncStream&lt;T&gt;} synchronized on
 	 *         {@code monitor} and backed by {@code stream}.
-	 * @throws NullPointerException
+	 * @throws NullPointerException if {@code stream == null}.
-	 *             if {@code stream == null}.
 	 */
 	public static <T> SyncStream<T> synchronizedStream(Stream<T> stream, Object monitor) {
 		Objects.requireNonNull(stream, "stream cannot be null");
@@ -1117,19 +1118,22 @@ public class SyncStreams {
 	}
 
 	/**
-	 * Wraps the given {@link IntStream} to make all <a href=
+	 * Wraps the given {@link IntStream} to make all
+	 * <a href=
 	 * "https://docs.oracle.com/javase/8/docs/api/java/util/stream/package-summary.html#StreamOps">
 	 * terminal operations</a> acquire the provided monitor's lock before
-	 * execution. Intermediate operations return streams that are also
+	 * execution. Intermediate operations
-	 * synchronized on the same object. The created stream will behave
+	 * return streams that are also synchronized on the same object. The created
-	 * identically to the provided stream in all other aspects. Use this to
+	 * stream will behave identically
-	 * synchronize access to stream's source.
+	 * to the provided stream in all other aspects. Use this to synchronize
+	 * access to stream's source.
 	 * <p>
 	 * <i>The returned {@code IntStream}'s {@link IntStream#iterator()
-	 * iterator()} and {@link IntStream#spliterator() spliterator()} methods
+	 * iterator()} and
-	 * return regular non-synchronized iterators and spliterators
+	 * {@link IntStream#spliterator() spliterator()} methods return regular
-	 * respectively</i>. It is the user's responsibility to avoid concurrency
+	 * non-synchronized iterators and
-	 * issues:
+	 * spliterators respectively</i>. It is the user's responsibility to avoid
+	 * concurrency issues:
 	 * 
 	 * <pre>
 	 * synchronized (stream.getMonitor()) {
@@ -1148,15 +1152,13 @@ public class SyncStreams {
 	 * stream.forEach(System.out::println); // Should never throw a ConcurrentModificationException
 	 * </pre>
 	 * 
-	 * @param stream
+	 * @param stream  the stream to wrap.
-	 *            the stream to wrap.
+	 * @param monitor the object that the stream will use for synchronization.
-	 * @param monitor
+	 *                When {@code null}, the stream
-	 *            the object that the stream will use for synchronization. When
+	 *                will synchronize on itself.
-	 *            {@code null}, the stream will synchronize on itself.
 	 * @return a {@link SyncIntStream} synchronized on {@code monitor} and
 	 *         backed by {@code stream}.
-	 * @throws NullPointerException
+	 * @throws NullPointerException if {@code stream == null}.
-	 *             if {@code stream == null}.
 	 */
 	public static SyncIntStream synchronizedStream(IntStream stream, Object monitor) {
 		Objects.requireNonNull(stream, "stream cannot be null");
@@ -1164,19 +1166,22 @@ public class SyncStreams {
 	}
 
 	/**
-	 * Wraps the given {@link LongStream} to make all <a href=
+	 * Wraps the given {@link LongStream} to make all
+	 * <a href=
 	 * "https://docs.oracle.com/javase/8/docs/api/java/util/stream/package-summary.html#StreamOps">
 	 * terminal operations</a> acquire the provided monitor's lock before
-	 * execution. Intermediate operations return streams that are also
+	 * execution. Intermediate operations
-	 * synchronized on the same object. The created stream will behave
+	 * return streams that are also synchronized on the same object. The created
-	 * identically to the provided stream in all other aspects. Use this to
+	 * stream will behave identically
-	 * synchronize access to stream's source.
+	 * to the provided stream in all other aspects. Use this to synchronize
+	 * access to stream's source.
 	 * <p>
 	 * <i>The returned {@code LongStream}'s {@link LongStream#iterator()
-	 * iterator()} and {@link LongStream#spliterator() spliterator()} methods
+	 * iterator()} and
-	 * return regular non-synchronized iterators and spliterators
+	 * {@link LongStream#spliterator() spliterator()} methods return regular
-	 * respectively</i>. It is the user's responsibility to avoid concurrency
+	 * non-synchronized iterators and
-	 * issues:
+	 * spliterators respectively</i>. It is the user's responsibility to avoid
+	 * concurrency issues:
 	 * 
 	 * <pre>
 	 * synchronized (stream.getMonitor()) {
@@ -1195,15 +1200,13 @@ public class SyncStreams {
 	 * stream.forEach(System.out::println); // Should never throw a ConcurrentModificationException
 	 * </pre>
 	 * 
-	 * @param stream
+	 * @param stream  the stream to wrap.
-	 *            the stream to wrap.
+	 * @param monitor the object that the stream will use for synchronization.
-	 * @param monitor
+	 *                When {@code null}, the stream
-	 *            the object that the stream will use for synchronization. When
+	 *                will synchronize on itself.
-	 *            {@code null}, the stream will synchronize on itself.
 	 * @return a {@link SyncLongStream} synchronized on {@code monitor} and
 	 *         backed by {@code stream}.
-	 * @throws NullPointerException
+	 * @throws NullPointerException if {@code stream == null}.
-	 *             if {@code stream == null}.
 	 */
 	public static SyncLongStream synchronizedStream(LongStream stream, Object monitor) {
 		Objects.requireNonNull(stream, "stream cannot be null");
@@ -1211,19 +1214,22 @@ public class SyncStreams {
 	}
 
 	/**
-	 * Wraps the given {@link DoubleStream} to make all <a href=
+	 * Wraps the given {@link DoubleStream} to make all
+	 * <a href=
 	 * "https://docs.oracle.com/javase/8/docs/api/java/util/stream/package-summary.html#StreamOps">
 	 * terminal operations</a> acquire the provided monitor's lock before
-	 * execution. Intermediate operations return streams that are also
+	 * execution. Intermediate operations
-	 * synchronized on the same object. The created stream will behave
+	 * return streams that are also synchronized on the same object. The created
-	 * identically to the provided stream in all other aspects. Use this to
+	 * stream will behave identically
-	 * synchronize access to stream's source.
+	 * to the provided stream in all other aspects. Use this to synchronize
+	 * access to stream's source.
 	 * <p>
 	 * <i>The returned {@code DoubleStream}'s {@link DoubleStream#iterator()
-	 * iterator()} and {@link DoubleStream#spliterator() spliterator()} methods
+	 * iterator()} and
-	 * return regular non-synchronized iterators and spliterators
+	 * {@link DoubleStream#spliterator() spliterator()} methods return regular
-	 * respectively</i>. It is the user's responsibility to avoid concurrency
+	 * non-synchronized iterators and
-	 * issues:
+	 * spliterators respectively</i>. It is the user's responsibility to avoid
+	 * concurrency issues:
 	 * 
 	 * <pre>
 	 * synchronized (stream.getMonitor()) {
@@ -1242,15 +1248,13 @@ public class SyncStreams {
 	 * stream.forEach(System.out::println); // Should never throw a ConcurrentModificationException
 	 * </pre>
 	 * 
-	 * @param stream
+	 * @param stream  the stream to wrap.
-	 *            the stream to wrap.
+	 * @param monitor the object that the stream will use for synchronization.
-	 * @param monitor
+	 *                When {@code null}, the stream
-	 *            the object that the stream will use for synchronization. When
+	 *                will synchronize on itself.
-	 *            {@code null}, the stream will synchronize on itself.
 	 * @return a {@link SyncDoubleStream} synchronized on {@code monitor} and
 	 *         backed by {@code stream}.
-	 * @throws NullPointerException
+	 * @throws NullPointerException if {@code stream == null}.
-	 *             if {@code stream == null}.
 	 */
 	public static SyncDoubleStream synchronizedStream(DoubleStream stream, Object monitor) {
 		Objects.requireNonNull(stream, "stream cannot be null");

src/main/java/ru/windcorp/jputil/SyntaxException.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil;
 
 public class SyntaxException extends Exception {

src/main/java/ru/windcorp/jputil/chars/CharArrayIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.text.CharacterIterator;
@@ -108,7 +108,7 @@ public class CharArrayIterator implements CharacterIterator {
 		return pos;
 	}
 
-	// @SuppressWarnings("all") Just STFU, this _is_ terrific
+//	@SuppressWarnings("all") Just STFU, this _is_ terrific
 
 	// SonarLint: "clone" should not be overridden (java:S2975)
 	// And I wouldn't have done that if only CharacterIterator had not required

src/main/java/ru/windcorp/jputil/chars/CharConsumer.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.util.function.IntConsumer;

src/main/java/ru/windcorp/jputil/chars/CharConsumers.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.util.Objects;

src/main/java/ru/windcorp/jputil/chars/CharPredicate.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.util.Arrays;

src/main/java/ru/windcorp/jputil/chars/CharSupplier.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.util.function.IntSupplier;

src/main/java/ru/windcorp/jputil/chars/EscapeException.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 public class EscapeException extends Exception {

src/main/java/ru/windcorp/jputil/chars/Escaper.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.text.CharacterIterator;
@@ -103,8 +103,14 @@ public class Escaper {
 
 	}
 
-	public static final Escaper JAVA = new Escaper('\\', 'u', "tbnrf'\"".toCharArray(), "\t\b\n\r\f\'\"".toCharArray(),
+	public static final Escaper JAVA = new Escaper(
-			true, true);
+		'\\',
+		'u',
+		"tbnrf'\"".toCharArray(),
+		"\t\b\n\r\f\'\"".toCharArray(),
+		true,
+		true
+	);
 
 	private final char escapeChar;
 	private final char unicodeEscapeChar;
@@ -114,8 +120,14 @@ public class Escaper {
 	private final boolean preferUnicode;
 	private final boolean strict;
 
-	protected Escaper(char escapeChar, char unicodeEscapeChar, char[] safes, char[] unsafes, boolean preferUnicode,
+	protected Escaper(
-			boolean strict) {
+		char escapeChar,
+		char unicodeEscapeChar,
+		char[] safes,
+		char[] unsafes,
+		boolean preferUnicode,
+		boolean strict
+	) {
 		this.escapeChar = escapeChar;
 		this.unicodeEscapeChar = unicodeEscapeChar;
 		this.safes = safes;
@@ -140,7 +152,8 @@ public class Escaper {
 		for (char c : unsafes) {
 			if (c == escapeChar)
 				throw new IllegalArgumentException(
-						"Unsafe characters contain escape chatacter (escape character is escaped automatically)");
+					"Unsafe characters contain escape chatacter (escape character is escaped automatically)"
+				);
 			if (c == unicodeEscapeChar)
 				throw new IllegalArgumentException("Unsafe characters contain Unicode escape chatacter");
 		}
@@ -160,7 +173,11 @@ public class Escaper {
 			end = Integer.MAX_VALUE;
 		else
 			end = src.getPosition() + length;
-		while (src.has() && src.getPosition() < end && (until == null || !until.test(src.current())))
+		while (
+			src.has() &&
+				src.getPosition() < end &&
+				(until == null || !until.test(src.current()))
+		)
 			escape(src.consume(), output);
 	}
 
@@ -208,7 +225,11 @@ public class Escaper {
 
 		int result = 0;
 
-		while (src.has() && src.getPosition() < end && (until == null || !until.test(src.current()))) {
+		while (
+			src.has() &&
+				src.getPosition() < end &&
+				(until == null || !until.test(src.current()))
+		) {
 			result += getEscapedLength(src.consume());
 		}
 
@@ -236,7 +257,11 @@ public class Escaper {
 			end = Integer.MAX_VALUE;
 		else
 			end = src.getPosition() + length;
-		while (src.has() && src.getPosition() < end && (until == null || !until.test(src.current()))) {
+		while (
+			src.has() &&
+				src.getPosition() < end &&
+				(until == null || !until.test(src.current()))
+		) {
 			output.accept(unescapeOneSequence(src));
 		}
 	}
@@ -257,8 +282,10 @@ public class Escaper {
 
 				if (src.current() == unicodeEscapeChar) {
 					src.next();
-					return (char) (hexValue(src.consume()) << (4 * 3) | hexValue(src.consume()) << (4 * 2)
+					return (char) (hexValue(src.consume()) << (4 * 3) |
-							| hexValue(src.consume()) << (4 * 1) | hexValue(src.consume()) << (4 * 0));
+						hexValue(src.consume()) << (4 * 2) |
+						hexValue(src.consume()) << (4 * 1) |
+						hexValue(src.consume()) << (4 * 0));
 				}
 
 				int index = ArrayUtil.firstIndexOf(safes, src.current());
@@ -288,7 +315,11 @@ public class Escaper {
 
 		int result = 0;
 
-		while (src.has() && src.getPosition() < end && (until == null || !until.test(src.current()))) {
+		while (
+			src.has() &&
+				src.getPosition() < end &&
+				(until == null || !until.test(src.current()))
+		) {
 			skipOneSequence(src);
 			result++;
 		}
@@ -297,7 +328,11 @@ public class Escaper {
 	}
 
 	public void skipOneSequence(CharReader src) {
-		if (src.current() == escapeChar && src.next() == unicodeEscapeChar) {
+		if (
+			src.current() == escapeChar
+				&&
+				src.next() == unicodeEscapeChar
+		) {
 			src.advance(4);
 		}
 		src.next();

src/main/java/ru/windcorp/jputil/chars/FancyCharacterIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.text.CharacterIterator;
@@ -86,7 +86,7 @@ public class FancyCharacterIterator implements CharacterIterator {
 		return sb.toString();
 	}
 
-	// @SuppressWarnings("all") Just STFU, this _is_ terrific
+//	@SuppressWarnings("all") Just STFU, this _is_ terrific
 
 	// SonarLint: "clone" should not be overridden (java:S2975)
 	// And I wouldn't have done that if only CharacterIterator had not required

src/main/java/ru/windcorp/jputil/chars/IndentedStringBuilder.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 public class IndentedStringBuilder {

src/main/java/ru/windcorp/jputil/chars/StringUtil.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.io.IOException;
@@ -41,8 +41,13 @@ public class StringUtil {
 	private static final String EMPTY_PLACEHOLDER = "[empty]";
 	private static final String DEFAULT_SEPARATOR = "; ";
 
-	public static <T> String arrayToString(T[] array, String separator, String empty, String nullPlaceholder,
+	public static <T> String arrayToString(
-			String nullArray) {
+		T[] array,
+		String separator,
+		String empty,
+		String nullPlaceholder,
+		String nullArray
+	) {
 
 		if (separator == null) {
 			throw new IllegalArgumentException(new NullPointerException());
@@ -74,8 +79,13 @@ public class StringUtil {
 		return arrayToString(array, DEFAULT_SEPARATOR);
 	}
 
-	public static String iteratorToString(Iterator<?> iterator, String separator, String empty, String nullPlaceholder,
+	public static String iteratorToString(
-			String nullIterator) {
+		Iterator<?> iterator,
+		String separator,
+		String empty,
+		String nullPlaceholder,
+		String nullIterator
+	) {
 
 		if (separator == null) {
 			throw new IllegalArgumentException(new NullPointerException());
@@ -109,8 +119,13 @@ public class StringUtil {
 		return iteratorToString(iterator, DEFAULT_SEPARATOR);
 	}
 
-	public static String iterableToString(Iterable<?> iterable, String separator, String empty, String nullPlaceholder,
+	public static String iterableToString(
-			String nullIterable) {
+		Iterable<?> iterable,
+		String separator,
+		String empty,
+		String nullPlaceholder,
+		String nullIterable
+	) {
 
 		if (separator == null) {
 			throw new IllegalArgumentException(new NullPointerException());
@@ -131,8 +146,14 @@ public class StringUtil {
 		return iterableToString(iterable, DEFAULT_SEPARATOR);
 	}
 
-	public static <T> String supplierToString(IntFunction<T> supplier, int length, String separator, String empty,
+	public static <T> String supplierToString(
-			String nullPlaceholder, String nullSupplier) {
+		IntFunction<T> supplier,
+		int length,
+		String separator,
+		String empty,
+		String nullPlaceholder,
+		String nullSupplier
+	) {
 
 		if (separator == null)
 			throw new IllegalArgumentException(new NullPointerException());
@@ -142,15 +163,28 @@ public class StringUtil {
 			return empty;
 
 		if (length > 0) {
-			return supplierToStringExactly(supplier, length, separator, nullPlaceholder);
+			return supplierToStringExactly(
+				supplier,
+				length,
+				separator,
+				nullPlaceholder
+			);
 		} else {
-			return supplierToStringUntilNull(supplier, separator, empty);
+			return supplierToStringUntilNull(
+				supplier,
+				separator,
+				empty
+			);
 		}
 
 	}
 
-	private static <T> String supplierToStringExactly(IntFunction<T> supplier, int length, String separator,
+	private static <T> String supplierToStringExactly(
-			String nullPlaceholder) {
+		IntFunction<T> supplier,
+		int length,
+		String separator,
+		String nullPlaceholder
+	) {
 		T element = supplier.apply(0);
 
 		StringBuilder sb = new StringBuilder(element == null ? nullPlaceholder : element.toString());
@@ -164,7 +198,11 @@ public class StringUtil {
 		return sb.toString();
 	}
 
-	private static <T> String supplierToStringUntilNull(IntFunction<T> supplier, String separator, String empty) {
+	private static <T> String supplierToStringUntilNull(
+		IntFunction<T> supplier,
+		String separator,
+		String empty
+	) {
 		T element = supplier.apply(0);
 
 		if (element == null) {
@@ -328,7 +366,11 @@ public class StringUtil {
 		StringBuilder sb = new StringBuilder();
 
 		charLoop: for (char c : src.toCharArray()) {
-			if ((resultIndex + 1) < arrayLength && test.test(c)) {
+			if (
+				(resultIndex + 1) < arrayLength
+					&&
+					test.test(c)
+			) {
 				result[resultIndex] = resetStringBuilder(sb);
 				++resultIndex;
 				continue charLoop;
@@ -347,17 +389,17 @@ public class StringUtil {
 	 * index.
 	 * <p>
 	 * Indices {@code 0} and {@code src.length() - 1} produce {@code str}
-	 * excluding the specified character and {@code ""}.
+	 * excluding
+	 * the specified character and {@code ""}.
 	 * <p>
 	 * 
-	 * @param src
+	 * @param src the String to split
-	 *            the String to split
+	 * @param at  index to split at
-	 * @param at
+	 * @throws IllegalArgumentException if the index is out of bounds for
-	 *            index to split at
+	 *                                  {@code src}
-	 * @throws IllegalArgumentException
-	 *             if the index is out of bounds for {@code src}
 	 * @return an array containing the substrings, in order of encounter in
-	 *         {@code src}. Its length is always 2.
+	 *         {@code src}.
+	 *         Its length is always 2.
 	 */
 	public static String[] splitAt(String src, int at) {
 		Objects.requireNonNull(src, "src");
@@ -374,7 +416,10 @@ public class StringUtil {
 			return new String[] { src.substring(0, src.length() - 1), "" };
 		}
 
-		return new String[] { src.substring(0, at), src.substring(at + 1) };
+		return new String[] {
+			src.substring(0, at),
+			src.substring(at + 1)
+		};
 	}
 
 	/**
@@ -382,7 +427,8 @@ public class StringUtil {
 	 * indices.
 	 * <p>
 	 * Indices {@code 0} and {@code src.length() - 1} produce extra zero-length
-	 * outputs. Duplicate indices produce extra zero-length outputs.
+	 * outputs.
+	 * Duplicate indices produce extra zero-length outputs.
 	 * <p>
 	 * Examples:
 	 * 
@@ -393,14 +439,13 @@ public class StringUtil {
 	 * splitAt("a.b",   1, 1, 1) -> {"a", "", "", "b"}
 	 * </pre>
 	 * 
-	 * @param src
+	 * @param src the String to split
-	 *            the String to split
+	 * @param at  indices to split at, in any order
-	 * @param at
+	 * @throws IllegalArgumentException if some index is out of bounds for
-	 *            indices to split at, in any order
+	 *                                  {@code src}
-	 * @throws IllegalArgumentException
-	 *             if some index is out of bounds for {@code src}
 	 * @return an array containing the substrings, in order of encounter in
-	 *         {@code src}. Its length is always {@code at.length + 1}.
+	 *         {@code src}.
+	 *         Its length is always {@code at.length + 1}.
 	 */
 	public static String[] splitAt(String src, int... at) {
 		Objects.requireNonNull(src, "src");
@@ -508,8 +553,10 @@ public class StringUtil {
 		}
 
 		if (endPos < beginPos) {
-			throw new IllegalArgumentException("endPos must be greater than or equal to beginPos (endPos=" + endPos
+			throw new IllegalArgumentException(
-					+ ", beginPos=" + beginPos + ")");
+				"endPos must be greater than or equal to beginPos (endPos="
+					+ endPos + ", beginPos=" + beginPos + ")"
+			);
 		}
 
 		if (endPos >= Math.min(a.length, b.length)) {
@@ -545,7 +592,8 @@ public class StringUtil {
 
 	/**
 	 * Finds and returns the index of the specified appearance of the specified
-	 * character in the given array. The search starts at index 0.
+	 * character
+	 * in the given array. The search starts at index 0.
 	 * <p>
 	 * Examples:
 	 * <p>
@@ -582,12 +630,10 @@ public class StringUtil {
 	 * </tr>
 	 * </table>
 	 * 
-	 * @param src
+	 * @param src    - the array to search in.
-	 *            - the array to search in.
+	 * @param target - the character to search for.
-	 * @param target
+	 * @param skip   - the amount of <code>target</code> characters to be
-	 *            - the character to search for.
+	 *               skipped.
-	 * @param skip
-	 *            - the amount of <code>target</code> characters to be skipped.
 	 * @return The index of the <code>skip+1</code>th <code>target</code>
 	 *         character or -1, if none found.
 	 * @see StringUtil#indexFromEnd(char[], char, int)
@@ -607,7 +653,8 @@ public class StringUtil {
 
 	/**
 	 * Finds and returns the index of the specified appearance of the specified
-	 * character in the given array. The search starts at index
+	 * character
+	 * in the given array. The search starts at index
 	 * <code>src.length - 1</code>.
 	 * <p>
 	 * Examples:
@@ -645,15 +692,13 @@ public class StringUtil {
 	 * </tr>
 	 * </table>
 	 * 
-	 * @param src
+	 * @param src    - the array to search in.
-	 *            - the array to search in.
+	 * @param target - the character to search for.
-	 * @param target
+	 * @param skip   - the amount of <code>target</code> characters to be
-	 *            - the character to search for.
+	 *               skipped.
-	 * @param skip
-	 *            - the amount of <code>target</code> characters to be skipped.
 	 * @return The index of the <code>skip+1</code>th
-	 *         <code>target</code>character from the end of the array or -1, if
+	 *         <code>target</code>character
-	 *         none found.
+	 *         from the end of the array or -1, if none found.
 	 * @see StringUtil#indexFromBeginning(char[], char, int)
 	 */
 	public static int indexFromEnd(char[] src, char target, int skip) {
@@ -828,8 +873,12 @@ public class StringUtil {
 		return result;
 	}
 
-	private static void buildCombinations(StringBuilder sb, Collection<String> result, Iterable<String>[] parts,
+	private static void buildCombinations(
-			int index) {
+		StringBuilder sb,
+		Collection<String> result,
+		Iterable<String>[] parts,
+		int index
+	) {
 		if (index >= parts.length) {
 			result.add(sb.toString());
 		} else {
@@ -855,8 +904,13 @@ public class StringUtil {
 		return result;
 	}
 
-	private static void buildCombinations(StringBuilder sb, String[] result, int[] resultIndex, String[][] parts,
+	private static void buildCombinations(
-			int index) {
+		StringBuilder sb,
+		String[] result,
+		int[] resultIndex,
+		String[][] parts,
+		int index
+	) {
 		if (index >= parts.length) {
 			result[resultIndex[0]++] = sb.toString();
 		} else {
@@ -931,7 +985,10 @@ public class StringUtil {
 	}
 
 	private static char hexDigit(long value, int digit) {
-		return hexDigit((int) (value >>> (4 * digit)) & 0xF);
+		return hexDigit(
+			(int) (value >>> (4 * digit))
+				& 0xF
+		);
 	}
 
 	public static char hexDigit(int value) {

src/main/java/ru/windcorp/jputil/chars/UncheckedEscapeException.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 public class UncheckedEscapeException extends RuntimeException {

src/main/java/ru/windcorp/jputil/chars/WordReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars;
 
 import java.io.IOException;

src/main/java/ru/windcorp/jputil/chars/reader/AbstractCharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 /**
@@ -27,9 +27,10 @@ public abstract class AbstractCharReader implements CharReader {
 
 	/**
 	 * Current position of this CharReader. The reader maps its input to
-	 * positions starting from 0. Positions that are negative or lower than 0
+	 * positions starting from 0.
-	 * are invalid. {@link #current()} will throw an exception if position is
+	 * Positions that are negative or lower than 0 are invalid.
-	 * invalid.
+	 * {@link #current()}
+	 * will throw an exception if position is invalid.
 	 */
 	protected int position = 0;
 

src/main/java/ru/windcorp/jputil/chars/reader/ArrayCharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 import java.util.Objects;

src/main/java/ru/windcorp/jputil/chars/reader/BufferedCharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 /**
@@ -51,12 +51,9 @@ public abstract class BufferedCharReader extends AbstractCharReader {
 	/**
 	 * Acquires next characters and stores them in the array.
 	 * 
-	 * @param buffer
+	 * @param buffer the output array
-	 *            the output array
+	 * @param offset index of the first character
-	 * @param offset
+	 * @param length maximum amount of characters to be pulled
-	 *            index of the first character
-	 * @param length
-	 *            maximum amount of characters to be pulled
 	 * @return the amount of characters actually pulled
 	 */
 	protected int pullChars(char[] buffer, int offset, int length) {

src/main/java/ru/windcorp/jputil/chars/reader/CharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 import java.io.IOException;
@@ -30,7 +30,7 @@ import ru.windcorp.jputil.chars.Escaper;
  */
 
 // SonarLint: Constants should not be defined in interfaces (java:S1214)
-// DONE is an essential part of the interface
+//   DONE is an essential part of the interface
 @SuppressWarnings("squid:S1214")
 
 public interface CharReader {
@@ -179,7 +179,8 @@ public interface CharReader {
 
 	/**
 	 * Skips to the end of the current line. Both <code>"\n"</code>,
-	 * <code>"\r"</code> and <code>"\r\n"</code> are considered line separators.
+	 * <code>"\r"</code>
+	 * and <code>"\r\n"</code> are considered line separators.
 	 * 
 	 * @return the amount of characters in the skipped line
 	 */

src/main/java/ru/windcorp/jputil/chars/reader/CharReaders.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 import java.io.InputStream;

src/main/java/ru/windcorp/jputil/chars/reader/ReaderCharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 import java.io.IOException;

src/main/java/ru/windcorp/jputil/chars/reader/StringCharReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.chars.reader;
 
 import java.util.Objects;
@@ -38,7 +38,8 @@ public class StringCharReader extends AbstractCharReader {
 		int end = offset + length;
 		if (end > str.length() || offset < 0)
 			throw new IllegalArgumentException(
-					"String contains [0; " + str.length() + "), requested [" + offset + "; " + end + ")");
+				"String contains [0; " + str.length() + "), requested [" + offset + "; " + end + ")"
+			);
 
 		this.offset = offset;
 		this.length = length;

src/main/java/ru/windcorp/jputil/functions/FloatSupplier.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 @FunctionalInterface

src/main/java/ru/windcorp/jputil/functions/ThrowingBiConsumer.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 import java.util.function.BiConsumer;
@@ -44,16 +44,9 @@ public interface ThrowingBiConsumer<T, U, E extends Exception> {
 	}
 
 	public static <T, U, E extends Exception> ThrowingBiConsumer<T, U, E> concat(
-			ThrowingBiConsumer<? super T, ? super U, ? extends E> first,
+		ThrowingBiConsumer<? super T, ? super U, ? extends E> first,
-			ThrowingBiConsumer<? super T, ? super U, ? extends E> second) {
+		ThrowingBiConsumer<? super T, ? super U, ? extends E> second
-		return (t, u) -> {
+	) {
-			first.accept(t, u);
-			second.accept(t, u);
-		};
-	}
-
-	public static <T, U, E extends Exception> ThrowingBiConsumer<T, U, E> concat(BiConsumer<? super T, ? super U> first,
-			ThrowingBiConsumer<? super T, ? super U, E> second) {
 		return (t, u) -> {
 			first.accept(t, u);
 			second.accept(t, u);
@@ -61,7 +54,19 @@ public interface ThrowingBiConsumer<T, U, E extends Exception> {
 	}
 
 	public static <T, U, E extends Exception> ThrowingBiConsumer<T, U, E> concat(
-			ThrowingBiConsumer<? super T, ? super U, E> first, BiConsumer<? super T, ? super U> second) {
+		BiConsumer<? super T, ? super U> first,
+		ThrowingBiConsumer<? super T, ? super U, E> second
+	) {
+		return (t, u) -> {
+			first.accept(t, u);
+			second.accept(t, u);
+		};
+	}
+
+	public static <T, U, E extends Exception> ThrowingBiConsumer<T, U, E> concat(
+		ThrowingBiConsumer<? super T, ? super U, E> first,
+		BiConsumer<? super T, ? super U> second
+	) {
 		return (t, u) -> {
 			first.accept(t, u);
 			second.accept(t, u);

src/main/java/ru/windcorp/jputil/functions/ThrowingConsumer.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 import java.util.function.BiConsumer;
@@ -39,24 +39,30 @@ public interface ThrowingConsumer<T, E extends Exception> {
 		};
 	}
 
-	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(ThrowingConsumer<? super T, ? extends E> first,
+	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(
-			ThrowingConsumer<? super T, ? extends E> second) {
+		ThrowingConsumer<? super T, ? extends E> first,
+		ThrowingConsumer<? super T, ? extends E> second
+	) {
 		return t -> {
 			first.accept(t);
 			second.accept(t);
 		};
 	}
 
-	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(Consumer<? super T> first,
+	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(
-			ThrowingConsumer<? super T, ? extends E> second) {
+		Consumer<? super T> first,
+		ThrowingConsumer<? super T, ? extends E> second
+	) {
 		return t -> {
 			first.accept(t);
 			second.accept(t);
 		};
 	}
 
-	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(ThrowingConsumer<? super T, ? extends E> first,
+	public static <T, E extends Exception> ThrowingConsumer<T, E> concat(
-			Consumer<? super T> second) {
+		ThrowingConsumer<? super T, ? extends E> first,
+		Consumer<? super T> second
+	) {
 		return t -> {
 			first.accept(t);
 			second.accept(t);

src/main/java/ru/windcorp/jputil/functions/ThrowingFunction.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 import java.util.function.BiConsumer;
@@ -28,8 +28,10 @@ public interface ThrowingFunction<T, R, E extends Exception> {
 	R apply(T t) throws E;
 
 	@SuppressWarnings("unchecked")
-	default Function<T, R> withHandler(BiConsumer<? super T, ? super E> handler,
+	default Function<T, R> withHandler(
-			Function<? super T, ? extends R> value) {
+		BiConsumer<? super T, ? super E> handler,
+		Function<? super T, ? extends R> value
+	) {
 		return t -> {
 			try {
 				return apply(t);
@@ -56,18 +58,23 @@ public interface ThrowingFunction<T, R, E extends Exception> {
 	}
 
 	public static <T, R, I, E extends Exception> ThrowingFunction<T, R, E> compose(
-			ThrowingFunction<? super T, I, ? extends E> first,
+		ThrowingFunction<? super T, I, ? extends E> first,
-			ThrowingFunction<? super I, ? extends R, ? extends E> second) {
+		ThrowingFunction<? super I, ? extends R, ? extends E> second
-		return t -> second.apply(first.apply(t));
+	) {
-	}
-
-	public static <T, R, I, E extends Exception> ThrowingFunction<T, R, E> compose(Function<? super T, I> first,
-			ThrowingFunction<? super I, ? extends R, E> second) {
 		return t -> second.apply(first.apply(t));
 	}
 
 	public static <T, R, I, E extends Exception> ThrowingFunction<T, R, E> compose(
-			ThrowingFunction<? super T, I, E> first, Function<? super I, ? extends R> second) {
+		Function<? super T, I> first,
+		ThrowingFunction<? super I, ? extends R, E> second
+	) {
+		return t -> second.apply(first.apply(t));
+	}
+
+	public static <T, R, I, E extends Exception> ThrowingFunction<T, R, E> compose(
+		ThrowingFunction<? super T, I, E> first,
+		Function<? super I, ? extends R> second
+	) {
 		return t -> second.apply(first.apply(t));
 	}
 

src/main/java/ru/windcorp/jputil/functions/ThrowingRunnable.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 import java.util.function.Consumer;
@@ -38,8 +38,10 @@ public interface ThrowingRunnable<E extends Exception> {
 		};
 	}
 
-	public static <E extends Exception> ThrowingRunnable<E> concat(ThrowingRunnable<? extends E> first,
+	public static <E extends Exception> ThrowingRunnable<E> concat(
-			ThrowingRunnable<? extends E> second) {
+		ThrowingRunnable<? extends E> first,
+		ThrowingRunnable<? extends E> second
+	) {
 		return () -> {
 			first.run();
 			second.run();

src/main/java/ru/windcorp/jputil/functions/ThrowingSupplier.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.functions;
 
 import java.util.function.Consumer;

src/main/java/ru/windcorp/jputil/iterators/ArrayIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.iterators;
 
 import java.util.Iterator;

src/main/java/ru/windcorp/jputil/iterators/FunctionIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.iterators;
 
 import java.util.Iterator;

src/main/java/ru/windcorp/jputil/iterators/PeekingIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.iterators;
 
 import java.util.Iterator;

src/main/java/ru/windcorp/jputil/iterators/RangeIterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.iterators;
 
 import java.util.Iterator;
@@ -49,8 +49,10 @@ public class RangeIterator<E> implements Iterator<E> {
 	public E next() {
 		update();
 		if (nextIndex >= from + amount) {
-			throw new NoSuchElementException("RangeIterator about to retrieve element " + nextIndex
+			throw new NoSuchElementException(
-					+ " which exceeds upper boundary " + (from + amount));
+				"RangeIterator about to retrieve element " + nextIndex
+					+ " which exceeds upper boundary " + (from + amount)
+			);
 		}
 
 		E result = parent.next();

src/main/java/ru/windcorp/jputil/iterators/Reiterator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.iterators;
 
 import java.util.ArrayList;

src/main/java/ru/windcorp/jputil/selectors/AbstractSelectorOperator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 public abstract class AbstractSelectorOperator implements SelectorOperator {

src/main/java/ru/windcorp/jputil/selectors/NamedParameterizedSelector.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import ru.windcorp.jputil.SyntaxException;

src/main/java/ru/windcorp/jputil/selectors/NamedSelector.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import ru.windcorp.jputil.SyntaxException;

src/main/java/ru/windcorp/jputil/selectors/OperatorAnd.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/OperatorExclude.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/OperatorNot.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/OperatorOr.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/OperatorXor.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/PredicateWrapper.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.function.Predicate;

src/main/java/ru/windcorp/jputil/selectors/Selector.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.function.Predicate;

src/main/java/ru/windcorp/jputil/selectors/SelectorOperator.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.Deque;

src/main/java/ru/windcorp/jputil/selectors/SelectorSystem.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.jputil.selectors;
 
 import java.util.ArrayList;
@@ -37,7 +37,7 @@ public class SelectorSystem<T> {
 	private final Collection<Selector<T>> selectors = Collections.synchronizedCollection(new ArrayList<Selector<T>>());
 
 	private final Collection<SelectorOperator> operators = Collections
-			.synchronizedCollection(new ArrayList<SelectorOperator>());
+		.synchronizedCollection(new ArrayList<SelectorOperator>());
 
 	private String stackPrefix = null;
 

src/main/java/ru/windcorp/progressia/Progressia.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia;
 
 public class Progressia {

src/main/java/ru/windcorp/progressia/ProgressiaLauncher.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia;
 
 import ru.windcorp.progressia.common.util.crash.CrashReports;

src/main/java/ru/windcorp/progressia/Proxy.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia;
 
 public interface Proxy {

src/main/java/ru/windcorp/progressia/client/Client.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client;
 
 import ru.windcorp.progressia.client.comms.DefaultClientCommsListener;
@@ -24,7 +24,7 @@ import ru.windcorp.progressia.client.graphics.world.Camera;
 import ru.windcorp.progressia.client.graphics.world.EntityAnchor;
 import ru.windcorp.progressia.client.graphics.world.LocalPlayer;
 import ru.windcorp.progressia.client.world.WorldRender;
-import ru.windcorp.progressia.common.world.DefaultWorldData;
+import ru.windcorp.progressia.common.world.WorldData;
 import ru.windcorp.progressia.common.world.entity.EntityData;
 
 public class Client {
@@ -36,7 +36,7 @@ public class Client {
 
 	private final ServerCommsChannel comms;
 
-	public Client(DefaultWorldData world, ServerCommsChannel comms) {
+	public Client(WorldData world, ServerCommsChannel comms) {
 		this.world = new WorldRender(world, this);
 		this.comms = comms;
 
@@ -69,7 +69,11 @@ public class Client {
 			return;
 		}
 
-		getCamera().setAnchor(new EntityAnchor(getWorld().getEntityRenderable(entity)));
+		getCamera().setAnchor(
+			new EntityAnchor(
+				getWorld().getEntityRenderable(entity)
+			)
+		);
 	}
 
 }

src/main/java/ru/windcorp/progressia/client/ClientProxy.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client;
 
 import ru.windcorp.progressia.Proxy;
@@ -42,8 +42,10 @@ public class ClientProxy implements Proxy {
 		try {
 			RenderTaskQueue.waitAndInvoke(FlatRenderProgram::init);
 			RenderTaskQueue.waitAndInvoke(WorldRenderProgram::init);
-			RenderTaskQueue.waitAndInvoke(() -> Typefaces
+			RenderTaskQueue.waitAndInvoke(
-					.setDefault(GNUUnifontLoader.load(ResourceManager.getResource("assets/unifont-13.0.03.hex.gz"))));
+				() -> Typefaces
+					.setDefault(GNUUnifontLoader.load(ResourceManager.getResource("assets/unifont-13.0.03.hex.gz")))
+			);
 		} catch (InterruptedException e) {
 			throw CrashReports.report(e, "ClientProxy failed");
 		}
@@ -58,7 +60,7 @@ public class ClientProxy implements Proxy {
 
 		ServerState.startServer();
 		ClientState.connectToLocalServer();
-
+		
 		TestMusicPlayer.start();
 	}
 

src/main/java/ru/windcorp/progressia/client/ClientState.java

@@ -15,13 +15,13 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client;
 
 import ru.windcorp.progressia.client.comms.localhost.LocalServerCommsChannel;
 import ru.windcorp.progressia.client.graphics.GUI;
 import ru.windcorp.progressia.client.graphics.world.LayerWorld;
-import ru.windcorp.progressia.common.world.DefaultWorldData;
+import ru.windcorp.progressia.common.world.WorldData;
 import ru.windcorp.progressia.server.ServerState;
 import ru.windcorp.progressia.test.LayerAbout;
 import ru.windcorp.progressia.test.LayerTestUI;
@@ -41,9 +41,11 @@ public class ClientState {
 
 	public static void connectToLocalServer() {
 
-		DefaultWorldData world = new DefaultWorldData();
+		WorldData world = new WorldData();
 
-		LocalServerCommsChannel channel = new LocalServerCommsChannel(ServerState.getInstance());
+		LocalServerCommsChannel channel = new LocalServerCommsChannel(
+			ServerState.getInstance()
+		);
 
 		Client client = new Client(world, channel);
 

src/main/java/ru/windcorp/progressia/client/ProgressiaClientMain.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client;
 
 import ru.windcorp.progressia.ProgressiaLauncher;

src/main/java/ru/windcorp/progressia/client/audio/AudioFormat.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio;
 
 public enum AudioFormat {

src/main/java/ru/windcorp/progressia/client/audio/AudioManager.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio;
 
 import org.lwjgl.openal.*;
@@ -43,7 +43,10 @@ public class AudioManager {
 	private static Speaker musicSpeaker;
 
 	public static void initAL() {
-		String defaultDeviceName = alcGetString(0, ALC_DEFAULT_DEVICE_SPECIFIER);
+		String defaultDeviceName = alcGetString(
+			0,
+			ALC_DEFAULT_DEVICE_SPECIFIER
+		);
 
 		device = alcOpenDevice(defaultDeviceName);
 
@@ -72,7 +75,10 @@ public class AudioManager {
 				lastSoundIndex = 0;
 			}
 			speaker = soundSpeakers.get(lastSoundIndex);
-		} while (speaker.getState().equals(Speaker.State.PLAYING_LOOP));
+		} while (
+			speaker.getState()
+				.equals(Speaker.State.PLAYING_LOOP)
+		);
 		return speaker;
 	}
 

src/main/java/ru/windcorp/progressia/client/audio/AudioRegistry.java

@@ -21,9 +21,9 @@ import ru.windcorp.progressia.client.audio.backend.SoundType;
 import ru.windcorp.progressia.common.util.namespaces.NamespacedInstanceRegistry;
 
 public class AudioRegistry extends NamespacedInstanceRegistry<SoundType> {
-
+	
 	private static final AudioRegistry INSTANCE = new AudioRegistry();
-
+	
 	/**
 	 * @return the instance
 	 */

src/main/java/ru/windcorp/progressia/client/audio/AudioSystem.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio;
 
 import ru.windcorp.progressia.common.resource.ResourceManager;
@@ -29,7 +29,10 @@ public class AudioSystem {
 	}
 
 	static void loadAudioData() {
-		AudioManager.loadSound(ResourceManager.getResource("assets/sounds/block_destroy_clap.ogg"),
+		AudioManager.loadSound(
-				"Progressia:BlockDestroy", AudioFormat.MONO);
+			ResourceManager.getResource("assets/sounds/block_destroy_clap.ogg"),
+			"Progressia:BlockDestroy",
+			AudioFormat.MONO
+		);
 	}
 }

src/main/java/ru/windcorp/progressia/client/audio/Music.java

@@ -15,15 +15,18 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio;
 
 import glm.vec._3.Vec3;
 import ru.windcorp.progressia.client.audio.backend.SoundType;
 import ru.windcorp.progressia.client.audio.backend.Speaker;
 
-public class Music extends Sound {
+public class Music 
-
+	extends Sound {
+	
+	
+	
 	public Music(SoundType soundType, int timeLength, float pitch, float gain) {
 		super(soundType, timeLength, new Vec3(), new Vec3(), pitch, gain);
 	}
@@ -44,7 +47,7 @@ public class Music extends Sound {
 	protected Speaker initSpeaker() {
 		return AudioManager.initMusicSpeaker(soundType);
 	}
-
+	
 	@Override
 	public void setPosition(Vec3 position) {
 		throw new UnsupportedOperationException();

src/main/java/ru/windcorp/progressia/client/audio/Sound.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio;
 
 import glm.vec._3.Vec3;
@@ -29,9 +29,9 @@ public class Sound {
 	protected float pitch = 1.0f;
 	protected float gain = 1.0f;
 	protected int timeLength = 0;
-
+	
 	protected SoundType soundType;
-
+	
 	public Sound(SoundType soundType) {
 		this.soundType = soundType;
 	}
@@ -39,23 +39,37 @@ public class Sound {
 	public Sound(String id) {
 		this(AudioRegistry.getInstance().get(id));
 	}
-
+	
-	public Sound(String id, int timeLength, Vec3 position, Vec3 velocity, float pitch, float gain) {
+	public Sound(
+		String id,
+		int timeLength,
+		Vec3 position,
+		Vec3 velocity,
+		float pitch,
+		float gain
+	) {
 		this(id);
 		this.position = position;
 		this.velocity = velocity;
 		this.pitch = pitch;
 		this.gain = gain;
 	}
-
+	
-	public Sound(SoundType soundType, int timeLength, Vec3 position, Vec3 velocity, float pitch, float gain) {
+	public Sound(
+		SoundType soundType,
+		int timeLength,
+		Vec3 position,
+		Vec3 velocity,
+		float pitch,
+		float gain
+	) {
 		this(soundType);
 		this.position = position;
 		this.velocity = velocity;
 		this.pitch = pitch;
 		this.gain = gain;
 	}
-
+	
 	protected Speaker initSpeaker() {
 		return AudioManager.initSpeaker(soundType);
 	}
@@ -105,7 +119,7 @@ public class Sound {
 	public float getPitch() {
 		return pitch;
 	}
-
+	
 	public double getDuration() {
 		return soundType.getDuration();
 	}

src/main/java/ru/windcorp/progressia/client/audio/backend/AudioReader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio.backend;
 
 import org.lwjgl.BufferUtils;
@@ -39,7 +39,12 @@ public class AudioReader {
 
 		ShortBuffer rawAudio = decodeVorbis(resource, channelBuffer, rateBuffer);
 
-		return new SoundType(id, rawAudio, format, rateBuffer.get(0));
+		return new SoundType(
+			id,
+			rawAudio,
+			format,
+			rateBuffer.get(0)
+		);
 	}
 
 	public static SoundType readAsMono(Resource resource, String id) {
@@ -50,7 +55,15 @@ public class AudioReader {
 		return readAsSpecified(resource, id, AL_FORMAT_STEREO16);
 	}
 
-	private static ShortBuffer decodeVorbis(Resource dataToDecode, IntBuffer channelsBuffer, IntBuffer rateBuffer) {
+	private static ShortBuffer decodeVorbis(
-		return stb_vorbis_decode_memory(dataToDecode.readAsBytes(), channelsBuffer, rateBuffer);
+		Resource dataToDecode,
+		IntBuffer channelsBuffer,
+		IntBuffer rateBuffer
+	) {
+		return stb_vorbis_decode_memory(
+			dataToDecode.readAsBytes(),
+			channelsBuffer,
+			rateBuffer
+		);
 	}
 }

src/main/java/ru/windcorp/progressia/client/audio/backend/Listener.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio.backend;
 
 import glm.vec._3.Vec3;
@@ -55,8 +55,9 @@ public class Listener {
 		if (isInWorld) {
 
 			if (wasInWorld) {
-				velocity.set(camera.getLastAnchorPosition()).sub(position)
+				velocity.set(camera.getLastAnchorPosition()).sub(position).div(
-						.div((float) GraphicsInterface.getFrameLength());
+					(float) GraphicsInterface.getFrameLength()
+				);
 			} else {
 				// If !wasInWorld, previous position is nonsence. Assume 0.
 				velocity.set(0);
@@ -71,9 +72,9 @@ public class Listener {
 		}
 
 		/*
-		 * Only apply if there is a chance that params changed. This can only
+		 * Only apply if there is a chance that params changed.
-		 * happen if we are in world now (isInWorld) or we just left world
+		 * This can only happen if we are in world now (isInWorld) or we just
-		 * (wasInWorld, then we need to reset).
+		 * left world (wasInWorld, then we need to reset).
 		 */
 		if (isInWorld || wasInWorld) {
 			applyParams();
@@ -90,7 +91,17 @@ public class Listener {
 	private void applyParams() {
 		alListener3f(AL_POSITION, position.x, position.y, position.z);
 		alListener3f(AL_VELOCITY, velocity.x, velocity.y, velocity.z);
-		alListenerfv(AL_ORIENTATION, new float[] { oriAt.x, oriAt.y, oriAt.z, oriUp.x, oriUp.y, oriUp.z });
+		alListenerfv(
+			AL_ORIENTATION,
+			new float[] {
+				oriAt.x,
+				oriAt.y,
+				oriAt.z,
+				oriUp.x,
+				oriUp.y,
+				oriUp.z
+			}
+		);
 	}
 
 }

src/main/java/ru/windcorp/progressia/client/audio/backend/SoundType.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio.backend;
 
 import ru.windcorp.progressia.common.util.namespaces.Namespaced;
@@ -34,7 +34,12 @@ public class SoundType extends Namespaced {
 	private int audioBuffer;
 	private double duration;
 
-	public SoundType(String id, ShortBuffer rawAudio, int format, int sampleRate) {
+	public SoundType(
+		String id,
+		ShortBuffer rawAudio,
+		int format,
+		int sampleRate
+	) {
 		super(id);
 		this.rawAudio = rawAudio;
 		this.sampleRate = sampleRate;
@@ -51,7 +56,7 @@ public class SoundType extends Namespaced {
 	public void initSpeaker(Speaker speaker) {
 		speaker.setAudioData(audioBuffer);
 	}
-
+	
 	public double getDuration() {
 		return duration;
 	}

src/main/java/ru/windcorp/progressia/client/audio/backend/Speaker.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.audio.backend;
 
 import glm.vec._3.Vec3;
@@ -24,7 +24,9 @@ import static org.lwjgl.openal.AL11.*;
 public class Speaker {
 
 	public enum State {
-		NOT_PLAYING, PLAYING, PLAYING_LOOP
+		NOT_PLAYING,
+		PLAYING,
+		PLAYING_LOOP
 	}
 
 	// Buffers
@@ -47,7 +49,13 @@ public class Speaker {
 		setAudioData(audioData);
 	}
 
-	public Speaker(int audioData, Vec3 position, Vec3 velocity, float pitch, float gain) {
+	public Speaker(
+		int audioData,
+		Vec3 position,
+		Vec3 velocity,
+		float pitch,
+		float gain
+	) {
 		setAudioData(audioData);
 		setPosition(position);
 		setVelocity(velocity);
@@ -55,7 +63,12 @@ public class Speaker {
 		setGain(gain);
 	}
 
-	public Speaker(Vec3 position, Vec3 velocity, float pitch, float gain) {
+	public Speaker(
+		Vec3 position,
+		Vec3 velocity,
+		float pitch,
+		float gain
+	) {
 		setPosition(position);
 		setVelocity(velocity);
 		setPitch(pitch);

src/main/java/ru/windcorp/progressia/client/comms/DefaultClientCommsListener.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms;
 
 import java.io.IOException;
@@ -39,7 +39,9 @@ public class DefaultClientCommsListener implements CommsListener {
 	@Override
 	public void onPacketReceived(Packet packet) {
 		if (packet instanceof PacketAffectWorld) {
-			((PacketAffectWorld) packet).apply(getClient().getWorld().getData());
+			((PacketAffectWorld) packet).apply(
+				getClient().getWorld().getData()
+			);
 		} else if (packet instanceof PacketSetLocalPlayer) {
 			setLocalPlayer((PacketSetLocalPlayer) packet);
 		}

src/main/java/ru/windcorp/progressia/client/comms/ServerCommsChannel.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms;
 
 import ru.windcorp.progressia.common.comms.CommsChannel;

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTrigger.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import ru.windcorp.progressia.common.util.namespaces.Namespaced;

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTriggerInputBased.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import ru.windcorp.progressia.client.graphics.input.InputEvent;

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTriggerLambda.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import java.util.function.BiConsumer;
@@ -33,12 +33,17 @@ public class ControlTriggerLambda extends ControlTriggerInputBased {
 	private final Predicate<InputEvent> predicate;
 	private final BiConsumer<InputEvent, ControlData> dataWriter;
 
-	public ControlTriggerLambda(String id, Predicate<InputEvent> predicate,
+	public ControlTriggerLambda(
-			BiConsumer<InputEvent, ControlData> dataWriter) {
+		String id,
+		Predicate<InputEvent> predicate,
+		BiConsumer<InputEvent, ControlData> dataWriter
+	) {
 		super(id);
 
-		this.packetId = NamespacedUtil.getId(NamespacedUtil.getNamespace(id),
+		this.packetId = NamespacedUtil.getId(
-				"ControlKeyPress" + NamespacedUtil.getName(id));
+			NamespacedUtil.getNamespace(id),
+			"ControlKeyPress" + NamespacedUtil.getName(id)
+		);
 
 		this.predicate = predicate;
 		this.dataWriter = dataWriter;
@@ -49,7 +54,10 @@ public class ControlTriggerLambda extends ControlTriggerInputBased {
 		if (!predicate.test(event))
 			return null;
 
-		PacketControl packet = new PacketControl(packetId, ControlDataRegistry.getInstance().create(getId()));
+		PacketControl packet = new PacketControl(
+			packetId,
+			ControlDataRegistry.getInstance().create(getId())
+		);
 
 		dataWriter.accept(event, packet.getControl());
 

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTriggerLocalLambda.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import java.util.function.Consumer;
@@ -29,7 +29,11 @@ public class ControlTriggerLocalLambda extends ControlTriggerInputBased {
 	private final Predicate<InputEvent> predicate;
 	private final Consumer<InputEvent> action;
 
-	public ControlTriggerLocalLambda(String id, Predicate<InputEvent> predicate, Consumer<InputEvent> action) {
+	public ControlTriggerLocalLambda(
+		String id,
+		Predicate<InputEvent> predicate,
+		Consumer<InputEvent> action
+	) {
 		super(id);
 
 		this.predicate = predicate;

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTriggerRegistry.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import ru.windcorp.progressia.common.util.namespaces.NamespacedInstanceRegistry;

src/main/java/ru/windcorp/progressia/client/comms/controls/ControlTriggers.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import java.util.function.BiConsumer;
@@ -27,59 +27,122 @@ import ru.windcorp.progressia.common.comms.controls.ControlData;
 
 public class ControlTriggers {
 
-	public static ControlTriggerInputBased of(String id, BiConsumer<InputEvent, ControlData> dataWriter,
+	public static ControlTriggerInputBased of(
-			Predicate<InputEvent> predicate) {
+		String id,
+		BiConsumer<InputEvent, ControlData> dataWriter,
+		Predicate<InputEvent> predicate
+	) {
 		return new ControlTriggerLambda(id, predicate, dataWriter);
 	}
 
-	public static ControlTriggerInputBased of(String id, Consumer<ControlData> dataWriter,
+	public static ControlTriggerInputBased of(
-			Predicate<InputEvent> predicate) {
+		String id,
-		return of(id, (input, control) -> dataWriter.accept(control), predicate);
+		Consumer<ControlData> dataWriter,
+		Predicate<InputEvent> predicate
+	) {
+		return of(
+			id,
+			(input, control) -> dataWriter.accept(control),
+			predicate
+		);
 	}
 
-	public static ControlTriggerInputBased of(String id, Predicate<InputEvent> predicate) {
+	public static ControlTriggerInputBased of(
-		return of(id, (input, control) -> {
+		String id,
-		}, predicate);
+		Predicate<InputEvent> predicate
+	) {
+		return of(
+			id,
+			(input, control) -> {
+			},
+			predicate
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased of(String id, Class<I> inputType,
+	public static <I extends InputEvent> ControlTriggerInputBased of(
-			BiConsumer<I, ControlData> dataWriter, Predicate<I>... predicates) {
+		String id,
-		return of(id, createCheckedDataWriter(inputType, dataWriter),
+		Class<I> inputType,
-				createCheckedCompoundPredicate(inputType, predicates));
+		BiConsumer<I, ControlData> dataWriter,
+		Predicate<I>... predicates
+	) {
+		return of(
+			id,
+			createCheckedDataWriter(inputType, dataWriter),
+			createCheckedCompoundPredicate(inputType, predicates)
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased of(String id, Class<I> inputType,
+	public static <I extends InputEvent> ControlTriggerInputBased of(
-			Consumer<ControlData> dataWriter, Predicate<I>... predicates) {
+		String id,
-		return of(id, inputType, (input, control) -> dataWriter.accept(control), predicates);
+		Class<I> inputType,
+		Consumer<ControlData> dataWriter,
+		Predicate<I>... predicates
+	) {
+		return of(
+			id,
+			inputType,
+			(input, control) -> dataWriter.accept(control),
+			predicates
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased of(String id, Class<I> inputType,
+	public static <I extends InputEvent> ControlTriggerInputBased of(
-			Predicate<I>... predicates) {
+		String id,
-		return of(id, (input, control) -> {
+		Class<I> inputType,
-		}, createCheckedCompoundPredicate(inputType, predicates));
+		Predicate<I>... predicates
+	) {
+		return of(
+			id,
+			(input, control) -> {
+			},
+			createCheckedCompoundPredicate(inputType, predicates)
+		);
 	}
 
 	@SafeVarargs
-	public static ControlTriggerInputBased of(String id, BiConsumer<InputEvent, ControlData> dataWriter,
+	public static ControlTriggerInputBased of(
-			Predicate<InputEvent>... predicates) {
+		String id,
-		return of(id, InputEvent.class, dataWriter, predicates);
+		BiConsumer<InputEvent, ControlData> dataWriter,
+		Predicate<InputEvent>... predicates
+	) {
+		return of(
+			id,
+			InputEvent.class,
+			dataWriter,
+			predicates
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased of(String id, Consumer<ControlData> dataWriter,
+	public static <I extends InputEvent> ControlTriggerInputBased of(
-			Predicate<InputEvent>... predicates) {
+		String id,
-		return of(id, (input, control) -> dataWriter.accept(control), predicates);
+		Consumer<ControlData> dataWriter,
+		Predicate<InputEvent>... predicates
+	) {
+		return of(
+			id,
+			(input, control) -> dataWriter.accept(control),
+			predicates
+		);
 	}
 
 	@SafeVarargs
-	public static ControlTriggerInputBased of(String id, Predicate<InputEvent>... predicates) {
+	public static ControlTriggerInputBased of(
-		return of(id, InputEvent.class, (input, control) -> {
+		String id,
-		}, predicates);
+		Predicate<InputEvent>... predicates
+	) {
+		return of(
+			id,
+			InputEvent.class,
+			(input, control) -> {
+			},
+			predicates
+		);
 	}
-
+	
 	//
 	//
 	///
@@ -93,53 +156,101 @@ public class ControlTriggers {
 	//
 	//
 	//
-
+	
-	public static ControlTriggerInputBased localOf(String id, Consumer<InputEvent> action,
+	public static ControlTriggerInputBased localOf(
-			Predicate<InputEvent> predicate) {
+		String id,
+		Consumer<InputEvent> action,
+		Predicate<InputEvent> predicate
+	) {
 		return new ControlTriggerLocalLambda(id, predicate, action);
 	}
 
-	public static ControlTriggerInputBased localOf(String id, Runnable action, Predicate<InputEvent> predicate) {
+	public static ControlTriggerInputBased localOf(
-		return localOf(id, input -> action.run(), predicate);
+		String id,
+		Runnable action,
+		Predicate<InputEvent> predicate
+	) {
+		return localOf(
+			id,
+			input -> action.run(),
+			predicate
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased localOf(String id, Class<I> inputType,
+	public static <I extends InputEvent> ControlTriggerInputBased localOf(
-			Consumer<I> action, Predicate<I>... predicates) {
+		String id,
-		return localOf(id, createCheckedAction(inputType, action),
+		Class<I> inputType,
-				createCheckedCompoundPredicate(inputType, predicates));
+		Consumer<I> action,
+		Predicate<I>... predicates
+	) {
+		return localOf(
+			id,
+			createCheckedAction(inputType, action),
+			createCheckedCompoundPredicate(inputType, predicates)
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased localOf(String id, Class<I> inputType,
+	public static <I extends InputEvent> ControlTriggerInputBased localOf(
-			Runnable action, Predicate<I>... predicates) {
+		String id,
-		return localOf(id, inputType, input -> action.run(), predicates);
+		Class<I> inputType,
+		Runnable action,
+		Predicate<I>... predicates
+	) {
+		return localOf(
+			id,
+			inputType,
+			input -> action.run(),
+			predicates
+		);
 	}
 
 	@SafeVarargs
-	public static ControlTriggerInputBased localOf(String id, Consumer<InputEvent> action,
+	public static ControlTriggerInputBased localOf(
-			Predicate<InputEvent>... predicates) {
+		String id,
-		return localOf(id, InputEvent.class, action, predicates);
+		Consumer<InputEvent> action,
+		Predicate<InputEvent>... predicates
+	) {
+		return localOf(
+			id,
+			InputEvent.class,
+			action,
+			predicates
+		);
 	}
 
 	@SafeVarargs
-	public static <I extends InputEvent> ControlTriggerInputBased localOf(String id, Runnable action,
+	public static <I extends InputEvent> ControlTriggerInputBased localOf(
-			Predicate<InputEvent>... predicates) {
+		String id,
-		return of(id, input -> action.run(), predicates);
+		Runnable action,
+		Predicate<InputEvent>... predicates
+	) {
+		return of(
+			id,
+			input -> action.run(),
+			predicates
+		);
 	}
 
 	private static <I extends InputEvent> BiConsumer<InputEvent, ControlData> createCheckedDataWriter(
-			Class<I> inputType, BiConsumer<I, ControlData> dataWriter) {
+		Class<I> inputType,
+		BiConsumer<I, ControlData> dataWriter
+	) {
 		return (inputEvent, control) -> dataWriter.accept(inputType.cast(inputEvent), control);
 	}
-
+	
-	private static <I extends InputEvent> Consumer<InputEvent> createCheckedAction(Class<I> inputType,
+	private static <I extends InputEvent> Consumer<InputEvent> createCheckedAction(
-			Consumer<I> action) {
+		Class<I> inputType,
+		Consumer<I> action
+	) {
 		return inputEvent -> action.accept(inputType.cast(inputEvent));
 	}
 
-	private static <I extends InputEvent> Predicate<InputEvent> createCheckedCompoundPredicate(Class<I> inputType,
+	private static <I extends InputEvent> Predicate<InputEvent> createCheckedCompoundPredicate(
-			Predicate<I>[] predicates) {
+		Class<I> inputType,
+		Predicate<I>[] predicates
+	) {
 		return new CompoundCastPredicate<>(inputType, predicates);
 	}
 

src/main/java/ru/windcorp/progressia/client/comms/controls/InputBasedControls.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.controls;
 
 import ru.windcorp.progressia.client.Client;
@@ -32,7 +32,8 @@ public class InputBasedControls {
 		this.client = client;
 
 		this.controls = ControlTriggerRegistry.getInstance().values().stream()
-				.filter(ControlTriggerInputBased.class::isInstance).toArray(ControlTriggerInputBased[]::new);
+			.filter(ControlTriggerInputBased.class::isInstance)
+			.toArray(ControlTriggerInputBased[]::new);
 	}
 
 	public void handleInput(Input input) {

src/main/java/ru/windcorp/progressia/client/comms/localhost/LocalClient.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.localhost;
 
 import java.io.IOException;

src/main/java/ru/windcorp/progressia/client/comms/localhost/LocalServerCommsChannel.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.comms.localhost;
 
 import ru.windcorp.progressia.client.comms.ServerCommsChannel;
@@ -34,7 +34,11 @@ public class LocalServerCommsChannel extends ServerCommsChannel {
 	public void connect(String login) {
 		setState(State.CONNECTED);
 
-		this.localClient = new LocalClient(server.getClientManager().grabClientId(), login, this);
+		this.localClient = new LocalClient(
+			server.getClientManager().grabClientId(),
+			login,
+			this
+		);
 
 		server.getClientManager().addClient(localClient);
 	}

src/main/java/ru/windcorp/progressia/client/graphics/Colors.java

@@ -34,13 +34,7 @@ public class Colors {
 		DEBUG_BLUE = toVector(0xFF0000FF),
 		DEBUG_CYAN = toVector(0xFF00FFFF),
 		DEBUG_MAGENTA = toVector(0xFFFF00FF),
-		DEBUG_YELLOW = toVector(0xFFFFFF00),
+		DEBUG_YELLOW = toVector(0xFFFFFF00);
-	
-		LIGHT_GRAY = toVector(0xFFCBCBD0),
-		BLUE = toVector(0xFF37A2E6),
-		HOVER_BLUE = toVector(0xFFC3E4F7),
-		DISABLED_GRAY = toVector(0xFFE5E5E5),
-		DISABLED_BLUE = toVector(0xFFB2D8ED);
 
 	public static Vec4 toVector(int argb) {
 		return toVector(argb, new Vec4());

src/main/java/ru/windcorp/progressia/client/graphics/GUI.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics;
 
 import java.util.ArrayList;
@@ -24,7 +24,6 @@ import java.util.List;
 
 import com.google.common.eventbus.Subscribe;
 
-import ru.windcorp.progressia.client.graphics.backend.GraphicsInterface;
 import ru.windcorp.progressia.client.graphics.input.CursorEvent;
 import ru.windcorp.progressia.client.graphics.input.FrameResizeEvent;
 import ru.windcorp.progressia.client.graphics.input.InputEvent;
@@ -43,7 +42,7 @@ public class GUI {
 	}
 
 	private static final List<LayerStackModification> MODIFICATION_QUEUE = Collections
-			.synchronizedList(new ArrayList<>());
+		.synchronizedList(new ArrayList<>());
 
 	private static class ModifiableInput extends Input {
 		@Override
@@ -58,24 +57,15 @@ public class GUI {
 	}
 
 	public static void addBottomLayer(Layer layer) {
-		modify(layers -> {
+		modify(layers -> layers.add(layer));
-			layers.add(layer);
-			layer.onAdded();
-		});
 	}
 
 	public static void addTopLayer(Layer layer) {
-		modify(layers -> {
+		modify(layers -> layers.add(0, layer));
-			layers.add(0, layer);
-			layer.onAdded();
-		});
 	}
 
 	public static void removeLayer(Layer layer) {
-		modify(layers -> {
+		modify(layers -> layers.remove(layer));
-			layers.remove(layer);
-			layer.onRemoved();
-		});
 	}
 
 	private static void modify(LayerStackModification mod) {
@@ -88,33 +78,12 @@ public class GUI {
 
 	public static void render() {
 		synchronized (LAYERS) {
-			
+			MODIFICATION_QUEUE.forEach(action -> action.affect(LAYERS));
-			if (!MODIFICATION_QUEUE.isEmpty()) {
+			MODIFICATION_QUEUE.clear();
-				MODIFICATION_QUEUE.forEach(action -> action.affect(LAYERS));
+
-				MODIFICATION_QUEUE.clear();
-				
-				boolean isMouseCurrentlyCaptured = GraphicsInterface.isMouseCaptured();
-				Layer.CursorPolicy policy = Layer.CursorPolicy.REQUIRE;
-				
-				for (Layer layer : LAYERS) {
-					Layer.CursorPolicy currentPolicy = layer.getCursorPolicy();
-					
-					if (currentPolicy != Layer.CursorPolicy.INDIFFERENT) {
-						policy = currentPolicy;
-						break;
-					}
-				}
-				
-				boolean shouldCaptureMouse = (policy == Layer.CursorPolicy.FORBID);
-				if (shouldCaptureMouse != isMouseCurrentlyCaptured) {
-					GraphicsInterface.setMouseCaptured(shouldCaptureMouse);
-				}
-			}
-			
 			for (int i = LAYERS.size() - 1; i >= 0; --i) {
 				LAYERS.get(i).render();
 			}
-			
 		}
 	}
 

src/main/java/ru/windcorp/progressia/client/graphics/Layer.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics;
 
 import java.util.concurrent.atomic.AtomicBoolean;
@@ -30,52 +30,15 @@ public abstract class Layer {
 	private boolean hasInitialized = false;
 
 	private final AtomicBoolean isValid = new AtomicBoolean(false);
-	
-	/**
-	 * Represents various requests that a {@link Layer} can make regarding the
-	 * presence of a visible cursor. The value of the highest layer that is not
-	 * {@link #INDIFFERENT} is used.
-	 */
-	public static enum CursorPolicy {
-		/**
-		 * Require that a cursor is visible.
-		 */
-		REQUIRE,
-
-		/**
-		 * The {@link Layer} should not affect the presence or absence of a
-		 * visible cursor; lower layers should be consulted.
-		 */
-		INDIFFERENT,
-
-		/**
-		 * Forbid a visible cursor.
-		 */
-		FORBID
-	}
-	
-	private CursorPolicy cursorPolicy = CursorPolicy.INDIFFERENT;
 
 	public Layer(String name) {
 		this.name = name;
 	}
-	
-	public String getName() {
-		return name;
-	}
 
 	@Override
 	public String toString() {
 		return "Layer " + name;
 	}
-	
-	public CursorPolicy getCursorPolicy() {
-		return cursorPolicy;
-	}
-	
-	public void setCursorPolicy(CursorPolicy cursorPolicy) {
-		this.cursorPolicy = cursorPolicy;
-	}
 
 	void render() {
 		GraphicsInterface.startNextLayer();
@@ -115,13 +78,5 @@ public abstract class Layer {
 	protected int getHeight() {
 		return GraphicsInterface.getFrameHeight();
 	}
-	
-	protected void onAdded() {
-		// Do nothing
-	}
-	
-	protected void onRemoved() {
-		// Do nothing
-	}
 
 }

src/main/java/ru/windcorp/progressia/client/graphics/backend/FaceCulling.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import java.util.ArrayDeque;

src/main/java/ru/windcorp/progressia/client/graphics/backend/GraphicsBackend.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import glm.vec._2.i.Vec2i;
@@ -61,7 +61,7 @@ public class GraphicsBackend {
 	static void setOpenGLInitialized(boolean isOpenGLInitialized) {
 		GraphicsBackend.isOpenGLInitialized = isOpenGLInitialized;
 	}
-
+	
 	public static void initialize() {
 		startRenderThread();
 	}
@@ -159,14 +159,27 @@ public class GraphicsBackend {
 
 	public static void setFullscreen() {
 		GLFWVidMode vidmode = glfwGetVideoMode(glfwGetPrimaryMonitor());
-		glfwSetWindowMonitor(getWindowHandle(), glfwGetPrimaryMonitor(), 0, 0, vidmode.width(), vidmode.height(), 0);
+		glfwSetWindowMonitor(
+			getWindowHandle(),
+			glfwGetPrimaryMonitor(),
+			0,
+			0,
+			vidmode.width(),
+			vidmode.height(),
+			0);
 		isFullscreen = true;
 	}
 
 	public static void setWindowed() {
 		GLFWVidMode vidmode = glfwGetVideoMode(glfwGetPrimaryMonitor());
-		glfwSetWindowMonitor(getWindowHandle(), 0, (vidmode.width() - getFrameWidth()) / 2,
+		glfwSetWindowMonitor(
-				(vidmode.height() - getFrameHeight()) / 2, getFrameWidth(), getFrameHeight(), 0);
+			getWindowHandle(),
+			0,
+			(vidmode.width() - getFrameWidth()) / 2,
+			(vidmode.height() - getFrameHeight()) / 2,
+			getFrameWidth(),
+			getFrameHeight(),
+			0);
 		isFullscreen = false;
 	}
 
@@ -179,18 +192,4 @@ public class GraphicsBackend {
 		GLFWVidMode vidmode = glfwGetVideoMode(glfwGetPrimaryMonitor());
 		return vidmode.refreshRate();
 	}
-	
-	public static boolean isMouseCaptured() {
-		return glfwGetInputMode(windowHandle, GLFW_CURSOR) == GLFW_CURSOR_DISABLED;
-	}
-	
-	public static void setMouseCaptured(boolean capture) {
-		int mode = capture ? GLFW_CURSOR_DISABLED : GLFW_CURSOR_NORMAL;
-		glfwSetInputMode(windowHandle, GLFW_CURSOR, mode);
-		
-		if (!capture) {
-			glfwSetCursorPos(windowHandle, FRAME_SIZE.x / 2.0, FRAME_SIZE.y / 2.0);
-		}
-	}
-	
 }

src/main/java/ru/windcorp/progressia/client/graphics/backend/GraphicsInterface.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import glm.vec._2.i.Vec2i;
@@ -81,13 +81,5 @@ public class GraphicsInterface {
 		}
 		GraphicsBackend.setVSyncEnabled(GraphicsBackend.isVSyncEnabled());
 	}
-	
-	public static boolean isMouseCaptured() {
-		return GraphicsBackend.isMouseCaptured();
-	}
-	
-	public static void setMouseCaptured(boolean capture) {
-		GraphicsBackend.setMouseCaptured(capture);
-	}
 
 }

src/main/java/ru/windcorp/progressia/client/graphics/backend/InputHandler.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import org.lwjgl.glfw.GLFW;
@@ -49,7 +49,13 @@ public class InputHandler {
 
 	private static final ModifiableKeyEvent THE_KEY_EVENT = new ModifiableKeyEvent();
 
-	static void handleKeyInput(long window, int key, int scancode, int action, int mods) {
+	static void handleKeyInput(
+		long window,
+		int key,
+		int scancode,
+		int action,
+		int mods
+	) {
 		if (GraphicsBackend.getWindowHandle() != window)
 			return;
 		THE_KEY_EVENT.initialize(key, scancode, action, mods);
@@ -65,7 +71,12 @@ public class InputHandler {
 		}
 	}
 
-	static void handleMouseButtonInput(long window, int key, int action, int mods) {
+	static void handleMouseButtonInput(
+		long window,
+		int key,
+		int action,
+		int mods
+	) {
 		handleKeyInput(window, key, Integer.MAX_VALUE - key, action, mods);
 	}
 
@@ -86,7 +97,11 @@ public class InputHandler {
 
 	private static final ModifiableCursorMoveEvent THE_CURSOR_MOVE_EVENT = new ModifiableCursorMoveEvent();
 
-	static void handleMouseMoveInput(long window, double x, double y) {
+	static void handleMouseMoveInput(
+		long window,
+		double x,
+		double y
+	) {
 		if (GraphicsBackend.getWindowHandle() != window)
 			return;
 		y = GraphicsInterface.getFrameHeight() - y; // Flip y axis
@@ -116,7 +131,11 @@ public class InputHandler {
 
 	private static final ModifiableWheelScrollEvent THE_WHEEL_SCROLL_EVENT = new ModifiableWheelScrollEvent();
 
-	static void handleWheelScroll(long window, double xoffset, double yoffset) {
+	static void handleWheelScroll(
+		long window,
+		double xoffset,
+		double yoffset
+	) {
 		if (GraphicsBackend.getWindowHandle() != window)
 			return;
 		THE_WHEEL_SCROLL_EVENT.initialize(xoffset, yoffset);
@@ -143,7 +162,10 @@ public class InputHandler {
 	/*
 	 * NB: this is NOT a GLFW callback, the raw callback is in GraphicsBackend
 	 */
-	static void handleFrameResize(int width, int height) {
+	static void handleFrameResize(
+		int width,
+		int height
+	) {
 		THE_FRAME_RESIZE_EVENT.initialize(width, height);
 		dispatch(THE_FRAME_RESIZE_EVENT);
 	}

src/main/java/ru/windcorp/progressia/client/graphics/backend/InputTracker.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import glm.vec._2.d.Vec2d;
@@ -24,7 +24,10 @@ import gnu.trove.set.hash.TIntHashSet;
 
 public class InputTracker {
 
-	private static final Vec2d CURSOR_POSITION = new Vec2d(Double.NaN, Double.NaN);
+	private static final Vec2d CURSOR_POSITION = new Vec2d(
+		Double.NaN,
+		Double.NaN
+	);
 
 	private static final TIntSet PRESSED_KEYS = new TIntHashSet(256);
 

src/main/java/ru/windcorp/progressia/client/graphics/backend/LWJGLInitializer.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import static org.lwjgl.opengl.GL11.*;
@@ -65,8 +65,10 @@ class LWJGLInitializer {
 
 		GraphicsBackend.setWindowHandle(handle);
 
+		glfwSetInputMode(handle, GLFW_CURSOR, GLFW_CURSOR_DISABLED);
+
 		glfwMakeContextCurrent(handle);
-		glfwSwapInterval(0); // TODO: remove after config system is added
+		glfwSwapInterval(0);	// TODO: remove after config system is added
 	}
 
 	private static void positionWindow() {
@@ -92,10 +94,16 @@ class LWJGLInitializer {
 	private static void setupWindowCallbacks() {
 		long handle = GraphicsBackend.getWindowHandle();
 
-		glfwSetFramebufferSizeCallback(handle, GraphicsBackend::onFrameResized);
+		glfwSetFramebufferSizeCallback(
+			handle,
+			GraphicsBackend::onFrameResized
+		);
 
 		glfwSetKeyCallback(handle, InputHandler::handleKeyInput);
-		glfwSetMouseButtonCallback(handle, InputHandler::handleMouseButtonInput);
+		glfwSetMouseButtonCallback(
+			handle,
+			InputHandler::handleMouseButtonInput
+		);
 
 		glfwSetCursorPosCallback(handle, InputHandler::handleMouseMoveInput);
 

src/main/java/ru/windcorp/progressia/client/graphics/backend/OpenGLObjectTracker.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import java.lang.ref.PhantomReference;
@@ -34,13 +34,19 @@ public class OpenGLObjectTracker {
 	private static final ReferenceQueue<OpenGLDeletable> DELETE_QUEUE = new ReferenceQueue<>();
 
 	public synchronized static void register(OpenGLDeletable object, IntConsumer glDeleter) {
-		GLPhantomReference<OpenGLDeletable> glRef = new GLPhantomReference<>(object, DELETE_QUEUE, object.getHandle(),
+		GLPhantomReference<OpenGLDeletable> glRef = new GLPhantomReference<>(
-				glDeleter);
+			object,
+			DELETE_QUEUE,
+			object.getHandle(),
+			glDeleter
+		);
 		TO_DELETE.add(glRef);
 	}
 
 	public static void deleteAllObjects() {
-		for (GLPhantomReference<OpenGLDeletable> glRef : TO_DELETE) {
+		for (
+			GLPhantomReference<OpenGLDeletable> glRef : TO_DELETE
+		) {
 			glRef.clear();
 		}
 	}
@@ -69,16 +75,20 @@ public class OpenGLObjectTracker {
 		 * It is possible to create a phantom reference with a {@code null}
 		 * queue, but such a reference is completely useless: Its {@code get}
 		 * method will always return {@code null} and, since it does not have a
-		 * queue, it will never be enqueued.
+		 * queue,
+		 * it will never be enqueued.
 		 *
-		 * @param referent
+		 * @param referent the object the new phantom reference will refer to
-		 *            the object the new phantom reference will refer to
+		 * @param q        the queue with which the reference is to be
-		 * @param q
+		 *                 registered,
-		 *            the queue with which the reference is to be registered, or
+		 *                 or {@code null} if registration is not required
-		 *            {@code null} if registration is not required
 		 */
-		public GLPhantomReference(T referent, ReferenceQueue<? super T> q, int referentGLhandle,
+		public GLPhantomReference(
-				IntConsumer GLDeleter) {
+			T referent,
+			ReferenceQueue<? super T> q,
+			int referentGLhandle,
+			IntConsumer GLDeleter
+		) {
 			super(referent, q);
 			this.referentGLhandle = referentGLhandle;
 			this.GLDeleter = GLDeleter;

src/main/java/ru/windcorp/progressia/client/graphics/backend/RenderTaskQueue.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import ru.windcorp.jputil.functions.ThrowingRunnable;
@@ -41,7 +41,11 @@ public class RenderTaskQueue {
 		HANDLER.invokeNow(task);
 	}
 
-	public static <E extends Exception> void waitAndInvoke(ThrowingRunnable<E> task) throws InterruptedException, E {
+	public static <E extends Exception> void waitAndInvoke(
+		ThrowingRunnable<E> task
+	)
+		throws InterruptedException,
+		E {
 		HANDLER.waitAndInvoke(task);
 	}
 

src/main/java/ru/windcorp/progressia/client/graphics/backend/RenderThread.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import static org.lwjgl.glfw.GLFW.*;

src/main/java/ru/windcorp/progressia/client/graphics/backend/Usage.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import static org.lwjgl.opengl.GL15.GL_DYNAMIC_DRAW;
@@ -23,7 +23,9 @@ import static org.lwjgl.opengl.GL15.GL_STATIC_DRAW;
 import static org.lwjgl.opengl.GL15.GL_STREAM_DRAW;
 
 public enum Usage { // TODO add _COPY and _READ, pref. as another enum
-	STATIC(GL_STATIC_DRAW), DYNAMIC(GL_DYNAMIC_DRAW), STREAM(GL_STREAM_DRAW);
+	STATIC(GL_STATIC_DRAW),
+	DYNAMIC(GL_DYNAMIC_DRAW),
+	STREAM(GL_STREAM_DRAW);
 
 	private final int glCode;
 

src/main/java/ru/windcorp/progressia/client/graphics/backend/VertexBufferObject.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend;
 
 import static org.lwjgl.opengl.GL20.*;
@@ -28,7 +28,8 @@ import ru.windcorp.progressia.client.graphics.backend.OpenGLObjectTracker.OpenGL
 public class VertexBufferObject implements OpenGLDeletable {
 
 	public static enum BindTarget {
-		ARRAY(GL_ARRAY_BUFFER), ELEMENT_ARRAY(GL_ELEMENT_ARRAY_BUFFER);
+		ARRAY(GL_ARRAY_BUFFER),
+		ELEMENT_ARRAY(GL_ELEMENT_ARRAY_BUFFER);
 
 		private final int glCode;
 

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/CombinedShader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders;
 
 import ru.windcorp.progressia.common.resource.Resource;
@@ -32,7 +32,12 @@ public class CombinedShader extends Shader {
 		for (int i = 1; i < resources.length; ++i) {
 			if (ShaderType.guessByResourceName(resources[i]) != first) {
 				throw new IllegalArgumentException(
-						"Deduced shader types of " + resources[0] + " and " + resources[i] + " differ");
+					"Deduced shader types of "
+						+ resources[0]
+						+ " and "
+						+ resources[i]
+						+ " differ"
+				);
 			}
 		}
 
@@ -66,8 +71,19 @@ public class CombinedShader extends Shader {
 			if (contents.codePointAt(versionIndex) == '#') {
 				final String versionAnnotation = "#version ";
 
-				if (contents.regionMatches(versionIndex, versionAnnotation, 0, versionAnnotation.length())) {
+				if (
-					contents = contents.substring(versionIndex + versionAnnotation.length() + "120".length());
+					contents.regionMatches(
+						versionIndex,
+						versionAnnotation,
+						0,
+						versionAnnotation.length()
+					)
+				) {
+					contents = contents.substring(
+						versionIndex
+							+ versionAnnotation.length()
+							+ "120".length()
+					);
 				}
 
 			}

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/Program.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders;
 
 import static org.lwjgl.opengl.GL11.*;

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/Shader.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders;
 
 import static org.lwjgl.opengl.GL11.*;
@@ -57,7 +57,10 @@ public class Shader implements OpenGLDeletable {
 			if (resource.contains("fsh"))
 				return FRAGMENT;
 
-			throw new IllegalArgumentException("Cannot deduce shader type from resource name \"" + resource + "\"");
+			throw new IllegalArgumentException(
+				"Cannot deduce shader type from resource name \"" +
+					resource + "\""
+			);
 		}
 	}
 
@@ -87,7 +90,10 @@ public class Shader implements OpenGLDeletable {
 	}
 
 	public Shader(String resource) {
-		this(ShaderType.guessByResourceName(resource), getShaderResource(resource).readAsString());
+		this(
+			ShaderType.guessByResourceName(resource),
+			getShaderResource(resource).readAsString()
+		);
 	}
 
 	@Override

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/attributes/Attribute.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.attributes;
 
 import ru.windcorp.progressia.client.graphics.backend.shaders.Program;

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/attributes/AttributeVertexArray.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.attributes;
 
 import static org.lwjgl.opengl.GL11.*;
@@ -51,29 +51,104 @@ public class AttributeVertexArray extends Attribute {
 		}
 	}
 
-	public void set(int size, boolean normalized, int stride, ByteBuffer pointer) {
+	public void set(
-		glVertexAttribPointer(handle, size, GL_BYTE, normalized, stride, pointer);
+		int size,
+		boolean normalized,
+		int stride,
+		ByteBuffer pointer
+	) {
+		glVertexAttribPointer(
+			handle,
+			size,
+			GL_BYTE,
+			normalized,
+			stride,
+			pointer
+		);
 	}
 
-	public void set(int size, boolean normalized, int stride, FloatBuffer pointer) {
+	public void set(
-		glVertexAttribPointer(handle, size, GL_FLOAT, normalized, stride, pointer);
+		int size,
+		boolean normalized,
+		int stride,
+		FloatBuffer pointer
+	) {
+		glVertexAttribPointer(
+			handle,
+			size,
+			GL_FLOAT,
+			normalized,
+			stride,
+			pointer
+		);
 	}
 
-	public void set(int size, boolean normalized, int stride, IntBuffer pointer) {
+	public void set(
-		glVertexAttribPointer(handle, size, GL_INT, normalized, stride, pointer);
+		int size,
+		boolean normalized,
+		int stride,
+		IntBuffer pointer
+	) {
+		glVertexAttribPointer(
+			handle,
+			size,
+			GL_INT,
+			normalized,
+			stride,
+			pointer
+		);
 	}
 
-	public void set(int size, boolean normalized, int stride, ShortBuffer pointer) {
+	public void set(
-		glVertexAttribPointer(handle, size, GL_SHORT, normalized, stride, pointer);
+		int size,
+		boolean normalized,
+		int stride,
+		ShortBuffer pointer
+	) {
+		glVertexAttribPointer(
+			handle,
+			size,
+			GL_SHORT,
+			normalized,
+			stride,
+			pointer
+		);
 	}
 
-	public void set(int size, int type, boolean normalized, int stride, long pointer) {
+	public void set(
-		glVertexAttribPointer(handle, size, type, normalized, stride, pointer);
+		int size,
+		int type,
+		boolean normalized,
+		int stride,
+		long pointer
+	) {
+		glVertexAttribPointer(
+			handle,
+			size,
+			type,
+			normalized,
+			stride,
+			pointer
+		);
 	}
 
-	public void set(int size, int type, boolean normalized, int stride, VertexBufferObject vbo, long offset) {
+	public void set(
+		int size,
+		int type,
+		boolean normalized,
+		int stride,
+		VertexBufferObject vbo,
+		long offset
+	) {
 		glBindBuffer(GL_ARRAY_BUFFER, vbo.getHandle());
-		glVertexAttribPointer(handle, size, type, normalized, stride, offset);
+		glVertexAttribPointer(
+			handle,
+			size,
+			type,
+			normalized,
+			stride,
+			offset
+		);
 	}
 
 }

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/uniforms/Uniform.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.uniforms;
 
 import ru.windcorp.progressia.client.graphics.backend.shaders.Program;

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/uniforms/Uniform1Float.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.uniforms;
 
 import static org.lwjgl.opengl.GL20.*;

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/uniforms/Uniform1Int.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.uniforms;
 
 import static org.lwjgl.opengl.GL20.*;

src/main/java/ru/windcorp/progressia/client/graphics/backend/shaders/uniforms/Uniform2Float.java

@@ -15,7 +15,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  */
-
+ 
 package ru.windcorp.progressia.client.graphics.backend.shaders.uniforms;
 
 import static org.lwjgl.opengl.GL20.*;