Skip to content

Commit

Permalink
consistent return type of taxi/elements when passed an element
Browse files Browse the repository at this point in the history
  • Loading branch information
@marshallbrekka
marshallbrekka committed Jan 31, 2013
1 parent f8e535a commit 4194449
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/clj_webdriver/taxi.clj
View file
Expand Up @@ -230,7 +230,7 @@
([q] (elements *driver* q))
([driver q]
(if (element-like? q)
q
(list q)
(*finder-fn* driver q))))

;; ## Driver functions ##
Expand Down

1 comment on commit 4194449

@semperos
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pull request!

I know it's only a single item in that branch of the if, but the default *finder-fn* will return a lazy seq. I'd like not to have one branch of this if return a linked list and the other a lazy seq, so if you'd make that match I'd be happy to accept this. It's definitely a mistake on my part that it returns a single item instead of a sequence of items.

Please sign in to comment.