Summary: | crash when rightclicking the animation curves docker | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | wolthera <griffinvalley> |
Component: | Dockers | Assignee: | Eoin O'Neill <eoinoneill1991> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | freebox64, halla |
Priority: | NOR | ||
Version: | git master (please specify the git hash!) | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/krita/abd10fde65a9692c4a63d4995fcfabbf62a6878a | Version Fixed In: | |
Sentry Crash Report: | |||
Attachments: | (Video example) |
Description
wolthera
2018-10-06 11:37:54 UTC
No, it seems to be happening when I rightclick the timeline ???? Git commit b3a62eb6d7863de12d2f94091b48c5ec5de47c85 by Boudewijn Rempt. Committed on 06/10/2018 at 18:09. Pushed by rempt into branch 'master'. Since the places where this method is called check whether 0 is returned or not, it's wrong to assert here. M +1 -1 libs/image/kis_layer.cc M +4 -2 libs/image/kis_paint_device.cc M +1 -1 libs/image/kis_paint_device.h https://commits.kde.org/krita/b3a62eb6d7863de12d2f94091b48c5ec5de47c85 Git commit b8dae8a7476734b306c80b8e8680916af16a2000 by Boudewijn Rempt. Committed on 10/10/2018 at 07:39. Pushed by rempt into branch 'krita/4.1'. Since the places where this method is called check whether 0 is returned or not, it's wrong to assert here. (cherry picked from commit b3a62eb6d7863de12d2f94091b48c5ec5de47c85) M +1 -1 libs/image/kis_layer.cc M +4 -2 libs/image/kis_paint_device.cc M +1 -1 libs/image/kis_paint_device.h https://commits.kde.org/krita/b8dae8a7476734b306c80b8e8680916af16a2000 Created attachment 115551 [details]
(Video example)
(Krita 4.2.0-pre-alpha (git 4a0b033) - Ubuntu 14.04 LTS, Unity)
Hi all, possibly the same issue still affects the "Animation Curves" docker in the latest nightly build? (see video)
1. right-click over the timeline of the "Animation Curves" docker
Actual Results: assert abort
That's actually a completely different thing... Thread 1 (Thread 0x7f59f6e7b980 (LWP 32568)): [KCrash Handler] #6 0x00007f59ea4e3f67 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:55 #7 0x00007f59ea4e533a in __GI_abort () at abort.c:78 #8 0x00007f59eb17ef1c in QMessageLogger::fatal(char const*, ...) const () from /usr/lib64/libQt5Core.so.5 #9 0x00007f59f1b22e04 in kis_assert_common(char const*, char const*, int, bool, bool) () from /home/boud/dev/i-krita/lib64/libkritaglobal.so.18 #10 0x00007f59f1b2315e in kis_safe_assert_recoverable(char const*, char const*, int) () from /home/boud/dev/i-krita/lib64/libkritaglobal.so.18 #11 0x00007f59f6038867 in KisActionManager::safePopulateMenu(QMenu*, QString const&, KisActionManager*) () from /home/boud/dev/i-krita/lib64/libkritaui.so.18 #12 0x00007f59be98e025 in TimelineRulerHeader::mousePressEvent(QMouseEvent*) () from /home/boud/dev/i-krita/lib64/kritaplugins/kritaanimationdocker.so #13 0x00007f59ec338ca0 in QWidget::event(QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #14 0x00007f59ec3dc39e in QFrame::event(QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #15 0x00007f59ec53b12b in QAbstractItemView::viewportEvent(QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #16 0x00007f59ec549dd0 in QHeaderView::viewportEvent(QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #17 0x00007f59eb35276e in QCoreApplicationPrivate::sendThroughObjectEventFilters(QObject*, QEvent*) () from /usr/lib64/libQt5Core.so.5 #18 0x00007f59ec2fd185 in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #19 0x00007f59ec304718 in QApplication::notify(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #20 0x00007f59f6255e3b in KisApplication::notify(QObject*, QEvent*) () from /home/boud/dev/i-krita/lib64/libkritaui.so.18 #21 0x00007f59eb3528f5 in QCoreApplication::notifyInternal2(QObject*, QEvent*) () from /usr/lib64/libQt5Core.so.5 #22 0x00007f59ec3031e9 in QApplicationPrivate::sendMouseEvent(QWidget*, QMouseEvent*, QWidget*, QWidget*, QWidget**, QPointer<QWidget>&, bool) () from /usr/lib64/libQt5Widgets.so.5 #23 0x00007f59ec35149f in ?? () from /usr/lib64/libQt5Widgets.so.5 #24 0x00007f59ec353993 in ?? () from /usr/lib64/libQt5Widgets.so.5 #25 0x00007f59ec2fd1ac in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #26 0x00007f59ec303b52 in QApplication::notify(QObject*, QEvent*) () from /usr/lib64/libQt5Widgets.so.5 #27 0x00007f59f6255e3b in KisApplication::notify(QObject*, QEvent*) () from /home/boud/dev/i-krita/lib64/libkritaui.so.18 #28 0x00007f59eb3528f5 in QCoreApplication::notifyInternal2(QObject*, QEvent*) () from /usr/lib64/libQt5Core.so.5 #29 0x00007f59eb8eb74f in QGuiApplicationPrivate::processMouseEvent(QWindowSystemInterfacePrivate::MouseEvent*) () from /usr/lib64/libQt5Gui.so.5 #30 0x00007f59eb8ed855 in QGuiApplicationPrivate::processWindowSystemEvent(QWindowSystemInterfacePrivate::WindowSystemEvent*) () from /usr/lib64/libQt5Gui.so.5 #31 0x00007f59eb8cc7eb in QWindowSystemInterface::sendWindowSystemEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib64/libQt5Gui.so.5 #32 0x00007f59dfb3108b in ?? () from /usr/lib64/libQt5XcbQpa.so.5 #33 0x00007f59eb35108b in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib64/libQt5Core.so.5 #34 0x00007f59eb359770 in QCoreApplication::exec() () from /usr/lib64/libQt5Core.so.5 #35 0x000000000040af08 in main () Git commit 86039f8ce3b273c0cab35832d08a147bfdd12d6c by Eoin O'Neill. Committed on 12/10/2018 at 13:03. Pushed by eoinoneill into branch 'master'. Fixed safe-assert on right click context menu in animation curves docker. Note: There should be another way to reuse animation timeline headers without sharing menu behavior between dockers that utilize them. For now, simply checking for an action manager is a simple work around for avoiding invalid menu population inside of kis_anim_curves_docker. M +10 -0 plugins/dockers/animation/timeline_ruler_header.cpp https://commits.kde.org/krita/86039f8ce3b273c0cab35832d08a147bfdd12d6c Git commit abd10fde65a9692c4a63d4995fcfabbf62a6878a by Boudewijn Rempt, on behalf of Eoin O'Neill. Committed on 20/11/2018 at 08:21. Pushed by rempt into branch 'krita/4.1'. Fixed safe-assert on right click context menu in animation curves docker. Note: There should be another way to reuse animation timeline headers without sharing menu behavior between dockers that utilize them. For now, simply checking for an action manager is a simple work around for avoiding invalid menu population inside of kis_anim_curves_docker. M +10 -0 plugins/dockers/animation/timeline_ruler_header.cpp https://commits.kde.org/krita/abd10fde65a9692c4a63d4995fcfabbf62a6878a |