Project

General

Profile

Bug #1583

Regression in "Mouse wheel changes selected chat" causing crash

Added by oprypin 23 days ago. Updated 7 days ago.

Status:
Confirmed
Priority:
Normal
Category:
-
Target version:
Start date:
07/12/2020
Due date:
% Done:

0%

Estimated time:
Version:
0.13.0
OS:
Any

Description

On latest master (not in 0.13.1 release), if "Mouse wheel changes selected chat" is enabled, scrolling over the chat list in many situations leads to a crash. One of such situations is if you just closed a chat room that was active.

Video of repro: https://i.imgur.com/2iPiyIn.mp4

https://github.com/quassel/quassel/commit/a453c963cf1872e14c83adf1d40a31821c166805
is the first bad commit

Reproduce it:

  • cmake .
    make
    printf "[ItemViews]\nMouseWheelChangesBuffer=true" >quasselclient.conf
    ./quassel --configdir=.
  • Accept all default setup
  • Enable Settings > Interface > Chat & Nick Lists > Mouse wheel changes selected chat
    ^(already ensured by the command above)
  • /query yourself
  • Right click on yourself, Hide Chat(s) Permanently
  • Hover over the chat list, scroll the mouse wheel
    crash
quassel-scroll.webm (1.22 MB) quassel-scroll.webm video of repro oprypin, 07/12/2020 10:44 PM

History

#1 Updated by digitalcircuit 23 days ago

  • Assignee set to digitalcircuit
  • Target version set to 0.14.0

#2 Updated by digitalcircuit 18 days ago

This has been fixed in a pull request, client: Fix crash on switching buffers with no buffer selected #544 . Once merged, this issue should automatically be closed.

If you don't mind re-compiling with the above patch, would you confirm that this issue is fixed for you as well? Optionally, if you have a GitHub account, you can comment there as well.

#3 Updated by digitalcircuit 7 days ago

  • Status changed from New to Confirmed

Also available in: Atom PDF