search

CreativeInquiry / PEmbroider

Embroidery Library for Processing
Other
442 stars 28 forks source link

Clamp ranges on property setters #33

Closed golanlevin closed 4 years ago

golanlevin commented 4 years ago

It should be impossible to set a stitch length which is zero or negative. Other properties with setter functions should impose reasonable safeties as well.

LingDong- commented 4 years ago

Safeties for all numeric setters added: 3c58914368c4290fe76ac7fcd065de602f187c95

Also, beginCull() can now optionally take one parameter, that sets the CULL_SPACING for you, e.g. beginCull(7) which is slightly neater than E.CULL_SPACING=7; beginCull().

golanlevin commented 4 years ago

Great!