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.