Repository: PaperMC/Paper
File: paper-api/src/main/java/com/destroystokyo/paper/entity/Pathfinder.java (Java)

Code before:
    boolean canFloat();

    /**
     * Makes this pathfinder assume that the mob can float, or not.
     *
     * @param canFloat if the mob can float, or not
     */
    void setCanFloat(boolean canFloat);

    /**
     * Represents the result of a pathfinding calculation.
     */
    interface PathResult {

        /**
         * All currently calculated points to follow along the path to reach the destination location.
         * <p>
         * Will return points the entity has already moved past, see {@link #getNextPointIndex()}.
         *
         * @return List of points
         */
        List<Location> getPoints();

        /**
         * @return Returns the index of the current point along the points returned in {@link #getPoints()} the entity
         * is trying to reach. This value will be higher than the maximum index of {@link #getPoints()} if this path finding is done
         */
        int getNextPointIndex();

        /**
         * @return The next location in the path points the entity is trying to reach, or null if there is no next point
         */
        @Nullable Location getNextPoint();

        /**
         * @return The closest point the path can get to the target location
         */
        @Nullable Location getFinalPoint();

        /**
         * Checks whether the final point can be reached.
         *
         * @return whether the final point can be reached
         * @see #getFinalPoint()
         */
        boolean canReachFinalPoint();
    }
}


Review comment (bug):
That's wrong as per oracle convention, also i would rather not do cleanup of this nature until checkstyle is in, can you revert all your changes apart from the two overload changes.

Code after:
    boolean canFloat();

    /**
     * Makes this pathfinder assume that the mob can float, or not
     *
     * @param canFloat if the mob can float, or not
     */
    void setCanFloat(boolean canFloat);

    /**
     * Represents the result of a pathfinding calculation
     */
    interface PathResult {

        /**
         * All currently calculated points to follow along the path to reach the destination location
         * <p>
         * Will return points the entity has already moved past, see {@link #getNextPointIndex()}
         *
         * @return List of points
         */
        List<Location> getPoints();

        /**
         * @return Returns the index of the current point along the points returned in {@link #getPoints()} the entity
         * is trying to reach. This value will be higher than the maximum index of {@link #getPoints()} if this path finding is done.
         */
        int getNextPointIndex();

        /**
         * @return The next location in the path points the entity is trying to reach, or null if there is no next point
         */
        @Nullable Location getNextPoint();

        /**
         * @return The closest point the path can get to the target location
         */
        @Nullable Location getFinalPoint();

        /**
         * Checks whether the final point can be reached
         *
         * @return whether the final point can be reached
         * @see #getFinalPoint()
         */
        boolean canReachFinalPoint();
    }
}