Subject: bug#38013: [PATCH] Rectangular region selection with mouse

From: Eli Zaretskii <[email-masked]>
Date: Sat, 23 Nov 2019 13:57:24 +0200

> From: Mattias Engdegård <[email-masked]>
> Date: Mon, 18 Nov 2019 19:08:24 +0100
> Cc: [email-masked], [email-masked]
> 
> Here is an updated patch that makes the selection work better with characters that aren't a multiple of the standard font width. 

Thanks, a few minor comments:

> +@defvar fine-grained-mouse-movement

I think this variable's name should start with "mouse-".  Like
mouse-fine-grained-movement or something.

> +    (setq fine-grained-mouse-movement t)

What happens if this function signals an error? won't
fine-grained-mouse-movement be left at its non-nil value?