◊(Local Yarn Code "View Ticket")

The macros poly-branch-tag and poly-branch-func are redundant. The second one does everything that is needed. The current poly-branch-tag should be eliminated, and poly-branch-func should be renamed to poly-branch-tag.

Resolved in [6b60dc]. It turns out the macros are not redundant: one handles keyword arguments and one doesn’t. Maybe I knew this at one point and forgot. At any rate, the resolution was to change the names of the macros and change the way they are documented.

I could have gone with a single function to handle all cases, but my thought was that perhaps I would benefit by not using define-tag-function in places where I didn’t need keyword arguments. I have not benchmarked this or anything, but define-tag-function is itself a macro that uses make-keyword-procedure, which seems needlessly complicated and, surely (?), slower than just defineing a simple function when keyword arguments are not needed.