search

flatsurf / surface-dynamics

Teichmüller dynamics in SageMath
https://flatsurf.github.io/surface-dynamics/
GNU General Public License v2.0
11 stars 13 forks source link

more fixes suggested by cython-lint #72

Closed fchapoton closed 7 months ago

fchapoton commented 10 months ago

This is just fixing a few details in origami, as suggested by cython-lint. Typically unused variables, code formatting, etc

Found using cython-lint --ignore=E501,E261,E222,E265,E266,E127,E128,E231,E741 surface_dynamics

fchapoton commented 7 months ago

Should I add back this line cdef ms, mo ? Its removal was suggested by cython-lint.

It looks a bit strange to me, as it is not giving a precise C type.

saraedum commented 7 months ago

Should I add back this line cdef ms, mo ? Its removal was suggested by cython-lint.

It looks a bit strange to me, as it is not giving a precise C type.

I don't think we need these declarations.