~sircmpwn/aerc

Check for terminal before forwarding mouse event v1 PROPOSED

Greg Anders: 1
 Check for terminal before forwarding mouse event

 1 files changed, 1 insertions(+), 1 deletions(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~sircmpwn/aerc/patches/9061/mbox | git am -3
Learn more about email & git

[PATCH] Check for terminal before forwarding mouse event Export this patch

When viewing a message part with no available filter, clicking the UI
would cause a nil pointer dereference because the MouseEvent was passed
to the PartViewer's `term` field, which does not exist in the case of an
absent filter.
---
 widgets/msgviewer.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/widgets/msgviewer.go b/widgets/msgviewer.go
index 7cd5553..4d41923 100644
--- a/widgets/msgviewer.go
+++ b/widgets/msgviewer.go
@@ -336,7 +336,7 @@ func (ps *PartSwitcher) MouseEvent(localX int, localY int, event tcell.Event) {
 		case tcell.Button1:
 			height := len(ps.parts)
 			y := ps.height - height
-			if localY < y {
+			if localY < y && ps.parts[ps.selected].term != nil {
 				ps.parts[ps.selected].term.MouseEvent(localX, localY, event)
 			}
 			for i, _ := range ps.parts {
-- 
2.24.0
Thanks!

To git.sr.ht:~sircmpwn/aerc
   8a84830..3338dce  master -> master
View this thread in the archives