> Or perhaps I have it backwards and "l1" and "l2" need to be swapped in > that description. But the mere fact that there is any question about > that means that the function is poorly documented and perhaps poorly > named as well. For that matter, is there a good reason why l1/l2 > have those roles and not the reverse?
Consistency. I organized all xxx_closept_yyy(Point *result, xxx *l1, yyy *l2) functions in a way that they find the find the point on "l1".