summaryrefslogtreecommitdiffstats
path: root/lib.c
diff options
context:
space:
mode:
authorKaz Kylheku <kaz@kylheku.com>2017-11-15 06:19:49 -0800
committerKaz Kylheku <kaz@kylheku.com>2017-11-15 06:19:49 -0800
commit8230a637a32ce0c88ba1c51aeb6c2d5edcece109 (patch)
treef8580dade862468172873b42cd1aaf82b3676478 /lib.c
parent6f7c75ed060f3c824740cf0a471be4a235f2fe51 (diff)
downloadtxr-8230a637a32ce0c88ba1c51aeb6c2d5edcece109.tar.gz
txr-8230a637a32ce0c88ba1c51aeb6c2d5edcece109.tar.bz2
txr-8230a637a32ce0c88ba1c51aeb6c2d5edcece109.zip
find-if: optimized rewrite and hash support.
* lib.c (find_if): Function rewritten to use the seq_info sequence classification mechanism, for much better performance on vector-like objects. Also, supports hash tables just like find_max. * txr.1: Documentation updated regarding hash support of find-if.
Diffstat (limited to 'lib.c')
-rw-r--r--lib.c57
1 files changed, 48 insertions, 9 deletions
diff --git a/lib.c b/lib.c
index ab6e3beb..088281f8 100644
--- a/lib.c
+++ b/lib.c
@@ -8594,19 +8594,58 @@ val find_min(val seq, val testfun, val keyfun)
return find_max(seq, default_arg(testfun, less_f), keyfun);
}
-val find_if(val pred, val list, val key)
+val find_if(val pred, val seq, val key)
{
- key = default_arg(key, identity_f);
- list = nullify(list);
+ val keyfun = default_arg(key, identity_f);
+ seq_info_t si = seq_info(seq);
- gc_hint(list);
+ switch (si.kind) {
+ case SEQ_NIL:
+ break;
+ case SEQ_HASHLIKE:
+ {
+ val hiter = hash_begin(si.obj);
+ val cell;
- for (; list; list = cdr(list)) {
- val item = car(list);
- val subj = funcall1(key, item);
+ while ((cell = hash_next(hiter))) {
+ val key = funcall1(keyfun, cell);
+ if (funcall1(pred, key))
+ return cell;
+ }
- if (funcall1(pred, subj))
- return item;
+ break;
+ }
+ case SEQ_LISTLIKE:
+ {
+ gc_hint(seq);
+
+ for (seq = z(si.obj); seq; seq = cdr(seq)) {
+ val elt = car(seq);
+ val key = funcall1(keyfun, elt);
+ if (funcall1(pred, key))
+ return elt;
+ }
+
+ break;
+ }
+ case SEQ_VECLIKE:
+ {
+ val vec = si.obj;
+ val len = length(vec);
+ val i;
+
+ for (i = zero; lt(i, len); i = succ(i)) {
+ val elt = ref(vec, i);
+ val key = funcall1(keyfun, elt);
+ if (funcall1(pred, key))
+ return elt;
+ }
+
+ break;
+ }
+ case SEQ_NOTSEQ:
+ default:
+ uw_throwf(error_s, lit("find-if: unsupported object ~s"), seq, nao);
}
return nil;