Nice work, contributions are greatly appreciated. :)
Since you're asking for constructive feedback, two things come to mind:
It's a matter of taste, but IMHO it's schemier that
/descriptive-variable/ is preferable to /desc-var/, so in your case
/s-foo/ could easily be /string-foo/. However, I would drop the prefix
altogether, as Chicken supports renaming on import. So /string-foo/
would become /foo/ and any prefixing is left up to the user of your library.
I'm not certain what you intended by 'official', but if you meant that
you'd like to see it packaged with Chicken then perhaps you should
consider switching the license to Chicken's license. Mind you, GPL3 eggs
are happily accepted, but note that many folks won't touch them for
various reasons. However, if this is a port of an elisp library then
likely re-licensing is not an option.
Anyhow, thanks for the hard work and the contribution! And welcome to
Chicken-land, come join us on IRC if you get a chance.
Thanks,
-Dan
On 2/20/2013 9:36 AM, Nicholas Van Horn wrote:
I'm new to Chicken (and scheme for that matter), but I'm eager to
contribute. I've been working on several text processing projects for
which I've ported an Emacs-lisp string manipulation library to Chicken.
The "strings" module provides many convenient string manipulation
procedures (including new procedures not ported from the original
elisp library). Each procedure is documented with examples. Beyond
their immediate convenience, (I think) they present the user with a
consistent API for working with strings.
I am eager to share this as an "official" egg, but being new to scheme
I have likely violated common practices. I share the preliminary
module here in the hopes that someone will find it useful. I welcome
any feedback with respect to changes in implementation/behavior, as
well as missing functionality that you would like to see added.
The module is packaged to be distributed as an egg, but it is
currently only available at:
https://github.com/n3mo/strings
Thanks,
Nicholas Van Horn
_______________________________________________
Chicken-users mailing list
Chicken-users@nongnu.org
https://lists.nongnu.org/mailman/listinfo/chicken-users
_______________________________________________
Chicken-users mailing list
Chicken-users@nongnu.org
https://lists.nongnu.org/mailman/listinfo/chicken-users