search

vseloved / rutils

Radical Utilities for Common Lisp
Other
246 stars 36 forks source link

Make remove-empty-subseqs default to t in split* functions #45

Open djr7C4 opened 4 years ago

djr7C4 commented 4 years ago

This makes the code and the tutorial consistent. Moreover, using a value of nil for remove-empty-subseqs as rarely useful (as noted in the tutorial). This fixes #44.

vseloved commented 4 years ago

Sorry, I have put it aside to think about potentially broken backward compatibility. I still continue this thinking :( I agree that t is a more reasonable choice, but I'm afraid that the change will surely break someone's code. That's how it always happens. So, I'll add the change in the next major version update. :man_shrugging: