Mouse clicks causing crashes when `number`/`relativenumber` is used with a custom `statuscolumn` about neovim

[luukvbaal]

Crash introduced in #26891, which fixed another crash... I'll look into it.

strange empty columns can be seen

The 'statuscolumn' intentionally follows 'numberwidth', which is documented. I don't think that's strange?

[OXY2DEV]

The 'statuscolumn' intentionally follows 'numberwidth', which is documented. I don't think that's strange?

Yeah, when I first saw the issue occur I had a custom statuscolumn(which was also 4 columns wide) set. So, when I saw a window where the statuscolumn should be empty also taking 4 columns I thought maybe Neovim made all the statuscolumn take the same amount of width. Which is why I said that.

I will go edit my comment.

[luukvbaal]

Returning early in do_mouse() when click_col >= wp->w_statuscol_click_defs_size() seems appropriate. I.e. still consider it a statuscolumn click for other (non)side effects, but do not try to call the click func for area that was not part of the generated statuscolumn but padded instead.