Clojurians Log v2

@lee to explore this matter of behavior for out-of-bounds input, i've tried calling find-last-by-pos with some other values. here are some results (i'll repeat some of the set up as the previous values are not logged, but thanks to borkdude, the following should get logged). given:

(require '[rewrite-clj.zip :as rz])
(def a-form "(defn hi-fn\n  [x]\n  (+ x 1))")

(-> a-form (rz/of-string {:track-position? true}) (rz/find-last-by-pos [1 0]) rz/string)

(-> a-form (rz/of-string {:track-position? true}) (rz/find-last-by-pos [2 0]) rz/string)
(-> a-form (rz/of-string {:track-position? true}) (rz/find-last-by-pos [3 0]) rz/string) 

(-> a-form (rz/of-string {:track-position? true}) (rz/find-last-by-pos [0 0]) rz/string)
(-> a-form (rz/of-string {:track-position? true}) (rz/find-last-by-pos [4 0]) rz/string)

Thanks @sogaiu, I strongly agree that 1 based needs to be documented very clearly. When I was reviewing your examples, I did not remember it was 1 based and was scratching my head a bit. I’ll have a look at your other scenarios sometime soon, thanks very much for digging in.

Wait... [2 0] and [3 0] returning “\n” is not good. I thought that maybe there might be value in distinguishing on or past end of line with “\n” vs past end of form with nil. Given your examples, and the confusion they cause, I think we should throw when <= 0.

Ok thanks again @sogaiu, I have updated my spike branch to now throw when any pos is <= 0. I appreciate any further feedback/awkwardnesses you find with this function while you exercise it.