All package-private methods and fields should have at least `protected` modifier

[rust2]

Currently, It's very painful to extend ShapeDrawer class in almost any way without directly modifying source code because most of fields and classes are just package-private

[earlygrey]

Fair enough, what fields / methods did you have in mind and what functionality do you extend / override? Most of the classes are package-private anyway so there are only a few you can subclass.

[rust2]

I finally got to this. That time I tried to add method to draw arbitrary quad but failed at some point. Now, 7 months later, I finally made util method which lets me draw single-colored quad. With more access I think I could also push different colors for gradient.
Not sure if it's done right but I don't see any problem yet

public class PolygonUtil {
    public static final short[] quadIds = new short[]{
            0, 1, 2, 2, 1, 3
    };

    private static final float[] tmpArray = new float[8];

    public static void quad(ShapeDrawer drawer,
                            float topLeftX,
                            float topLeftY,
                            float topRightX,
                            float topRightY,
                            float bottomLeftX,
                            float bottomLeftY,
                            float bottomRightX,
                            float bottomRightY) {
        tmpArray[0] = topLeftX;
        tmpArray[1] = topLeftY;
        tmpArray[2] = topRightX;
        tmpArray[3] = topRightY;
        tmpArray[4] = bottomLeftX;
        tmpArray[5] = bottomLeftY;
        tmpArray[6] = bottomRightX;
        tmpArray[7] = bottomRightY;
        drawer.filledPolygon(tmpArray, quadIds);
    }
}