Hrvoje Senjan 2015-06-25 17:01:28 +00:00 committed by Git OBS Bridge
parent 28093ee57d
commit 926a3f3d14
7 changed files with 20 additions and 58 deletions

View File

@ -1,20 +0,0 @@
=== modified file 'src/dbusmenuimporter.cpp'
--- src/dbusmenuimporter.cpp 2015-05-05 19:42:58 +0000
+++ src/dbusmenuimporter.cpp 2015-05-05 19:54:13 +0000
@@ -571,12 +571,13 @@
if (needRefresh || menu->actions().isEmpty()) {
d->m_idsRefreshedByAboutToShow << id;
- watcher = d->refresh(id);
- if (!d->waitForWatcher(watcher, REFRESH_TIMEOUT)) {
+ QDBusPendingCallWatcher *watcher2 = d->refresh(id);
+ if (!d->waitForWatcher(watcher2, REFRESH_TIMEOUT)) {
#ifndef QT_NO_DEBUG
DMWARNING << "Application did not refresh before timeout";
#endif
}
+ //watcher2->deleteLater();
}
}

View File

@ -1,20 +0,0 @@
=== modified file 'src/dbusmenuimporter.cpp'
--- src/dbusmenuimporter.cpp 2014-03-05 09:53:39 +0000
+++ src/dbusmenuimporter.cpp 2015-05-05 19:42:58 +0000
@@ -433,6 +433,7 @@
void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher)
{
int parentId = watcher->property(DBUSMENU_PROPERTY_ID).toInt();
+ watcher->deleteLater();
QDBusPendingReply<uint, DBusMenuLayoutItem> reply = *watcher;
if (!reply.isValid()) {
@@ -538,6 +539,7 @@
void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher *watcher)
{
int id = watcher->property(DBUSMENU_PROPERTY_ID).toInt();
+ watcher->deleteLater();
QDBusPendingReply<bool> reply = *watcher;
if (reply.isError()) {

View File

@ -1,3 +1,11 @@
-------------------------------------------------------------------
Thu Jun 25 16:52:13 UTC 2015 - hrvoje.senjan@gmail.com
- Update to 0.9.3+15.10.20150604:
* Fixed memory leaks
- Drop Clean-up-variable-naming.patch and dont-leak-all-the-watchers.diff,
merged upstream
------------------------------------------------------------------- -------------------------------------------------------------------
Fri May 15 19:29:14 UTC 2015 - hrvoje.senjan@gmail.com Fri May 15 19:29:14 UTC 2015 - hrvoje.senjan@gmail.com

View File

@ -19,7 +19,7 @@
%define rname libdbusmenu-qt %define rname libdbusmenu-qt
Name: libdbusmenu-qt5 Name: libdbusmenu-qt5
Version: 0.9.3+14.10.20140619 Version: 0.9.3+15.10.20150604
Release: 0 Release: 0
Url: https://launchpad.net/libdbusmenu-qt/ Url: https://launchpad.net/libdbusmenu-qt/
Summary: A Qt implementation of the DBusMenu protocol Summary: A Qt implementation of the DBusMenu protocol
@ -32,10 +32,6 @@ Source1: baselibs.conf
Patch1: noqDebug-qWarnings.patch Patch1: noqDebug-qWarnings.patch
# PATCH-FIX-UPSTREAM full_include_dir.patch -- CMake 2.8.12 creates a fatal error on relative include dirs for a target. silence that policy # PATCH-FIX-UPSTREAM full_include_dir.patch -- CMake 2.8.12 creates a fatal error on relative include dirs for a target. silence that policy
Patch2: full_include_dir.patch Patch2: full_include_dir.patch
# PATCH-FIX-UPSTREAM dont-leak-all-the-watchers.diff
Patch3: dont-leak-all-the-watchers.diff
# PATCH-FIX-UPSTREAM Clean-up-variable-naming.patch
Patch4: Clean-up-variable-naming.patch
#Needed for DISABLE_FIND_PACKAGE #Needed for DISABLE_FIND_PACKAGE
BuildRequires: cmake >= 2.8.6 BuildRequires: cmake >= 2.8.6
BuildRequires: doxygen BuildRequires: doxygen
@ -78,8 +74,6 @@ This package contains development files for libdbusmenu-qt5.
%if 0%{?suse_version} <= 1310 %if 0%{?suse_version} <= 1310
%patch2 -p1 %patch2 -p1
%endif %endif
%patch3 -p0
%patch4 -p0
# Remove build time references so build-compare can do its work # Remove build time references so build-compare can do its work
sed -i "s/HTML_TIMESTAMP = YES/HTML_TIMESTAMP = NO/" Doxyfile.in sed -i "s/HTML_TIMESTAMP = YES/HTML_TIMESTAMP = NO/" Doxyfile.in

View File

@ -1,3 +0,0 @@
version https://git-lfs.github.com/spec/v1
oid sha256:e531b29388317f48e22c652c7111e9c7750aa7d429a72742c49a1c8af1e61968
size 46641

View File

@ -0,0 +1,3 @@
version https://git-lfs.github.com/spec/v1
oid sha256:dd87926003be6455420a077dd8edb29f5dac428b9b299b2bd73a1c29c2126328
size 46318

View File

@ -36,7 +36,7 @@ index f25718d..8868efb 100644
} }
QVariantMap map = propertiesForAction(action); QVariantMap map = propertiesForAction(action);
diff --git a/src/dbusmenuimporter.cpp b/src/dbusmenuimporter.cpp diff --git a/src/dbusmenuimporter.cpp b/src/dbusmenuimporter.cpp
index 2a4c514..2f03729 100644 index 39f5995..d629d69 100644
--- a/src/dbusmenuimporter.cpp --- a/src/dbusmenuimporter.cpp
+++ b/src/dbusmenuimporter.cpp +++ b/src/dbusmenuimporter.cpp
@@ -94,7 +94,9 @@ public: @@ -94,7 +94,9 @@ public:
@ -79,7 +79,7 @@ index 2a4c514..2f03729 100644
return false; return false;
} }
@@ -436,18 +444,24 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher) @@ -437,18 +445,24 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher)
QDBusPendingReply<uint, DBusMenuLayoutItem> reply = *watcher; QDBusPendingReply<uint, DBusMenuLayoutItem> reply = *watcher;
if (!reply.isValid()) { if (!reply.isValid()) {
@ -104,7 +104,7 @@ index 2a4c514..2f03729 100644
return; return;
} }
@@ -474,7 +488,9 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher) @@ -475,7 +489,9 @@ void DBusMenuImporter::slotGetLayoutFinished(QDBusPendingCallWatcher *watcher)
} }
} }
#ifdef BENCHMARK #ifdef BENCHMARK
@ -114,7 +114,7 @@ index 2a4c514..2f03729 100644
#endif #endif
} }
@@ -513,7 +529,9 @@ void DBusMenuImporter::slotMenuAboutToShow() @@ -514,7 +530,9 @@ void DBusMenuImporter::slotMenuAboutToShow()
QPointer<QObject> guard(this); QPointer<QObject> guard(this);
if (!d->waitForWatcher(watcher, ABOUT_TO_SHOW_TIMEOUT)) { if (!d->waitForWatcher(watcher, ABOUT_TO_SHOW_TIMEOUT)) {
@ -124,7 +124,7 @@ index 2a4c514..2f03729 100644
} }
#ifdef BENCHMARK #ifdef BENCHMARK
@@ -541,7 +559,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher * @@ -543,7 +561,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher *
QDBusPendingReply<bool> reply = *watcher; QDBusPendingReply<bool> reply = *watcher;
if (reply.isError()) { if (reply.isError()) {
@ -134,10 +134,10 @@ index 2a4c514..2f03729 100644
return; return;
} }
bool needRefresh = reply.argumentAt<0>(); bool needRefresh = reply.argumentAt<0>();
@@ -553,7 +573,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher * @@ -555,7 +575,9 @@ void DBusMenuImporter::slotAboutToShowDBusCallFinished(QDBusPendingCallWatcher *
d->m_idsRefreshedByAboutToShow << id; d->m_idsRefreshedByAboutToShow << id;
watcher = d->refresh(id); QDBusPendingCallWatcher *watcher2 = d->refresh(id);
if (!d->waitForWatcher(watcher, REFRESH_TIMEOUT)) { if (!d->waitForWatcher(watcher2, REFRESH_TIMEOUT)) {
+#ifndef QT_NO_DEBUG +#ifndef QT_NO_DEBUG
DMWARNING << "Application did not refresh before timeout"; DMWARNING << "Application did not refresh before timeout";
+#endif +#endif