There are a lot of methods in the TPen class that are defined but not used because all the child classes overwrite almost all the functions and TPen itself isn't used directly a whole lot. This patch removes the unused code so it's less confusing to read turtle.py.

While TPen doesn't seem to be documented, it's not prefixed by a '_', so there might people using it and relying on these methods.

Ah, I think I was mistaken about some things when I made the first patch but it looks like TPen have a few methods that don't work as you'd expect. Since _colorstr isn't implemented on TPen fillcolor and pencolor pen don't work as intended when you call them directly from TPen. Jesstess suggested raising a NotImplementedError on these functions. Would that be a good change to make?

Hi ingrid,

Thanks for your work on this issue.

I'm triaging the open "turtle" issues, and I'm going to go ahead and close this one. If you would like to open a new, more specific issue, I would welcome that. Thanks.

nosy: + willingc
messages: + msg319745

resolution: remind
stage: resolved