-
Notifications
You must be signed in to change notification settings - Fork 46
Open
Labels
category: buildenhancementhelp wantedPrimary maintainers may not have time to resolve thisPrimary maintainers may not have time to resolve thispriority: low
Description
As discussed in #37, one option to clean up the aggdraw code is to use cython. As mentioned in #43, I recently noticed that all of the classes in aggdraw are actually declared as factory functions instead of real classes. I don't think any of the aggdraw functionality requires this design so it would be nice if the classes were declared as real classes.
I think the best idea would be to convert the C++ file to cython. This should:
- Make the code easier to read
- Avoid compatibility issues with new versions of python
- Limit unicode/bytes issues and handling
I think this rewrite could wait until aggdraw 2.0 and could look at adding #3's functionality/updating the agg version.
Metadata
Metadata
Assignees
Labels
category: buildenhancementhelp wantedPrimary maintainers may not have time to resolve thisPrimary maintainers may not have time to resolve thispriority: low