Seems to be some confusion and inconvenience from having a few methods to write many different types of shapes, and having to set the shapetype with a constant, which is easy to not realize or forget (e.g. #149, #102).
Switch so there's only one method for each shapetype, no constants needed, allowing for better input validation as well.
Seems to be some confusion and inconvenience from having a few methods to write many different types of shapes, and having to set the shapetype with a constant, which is easy to not realize or forget (e.g. #149, #102).
Switch so there's only one method for each shapetype, no constants needed, allowing for better input validation as well.