> On April 5, 2015, 5:54 p.m., Lamarque Souza wrote: > > src/solid/devices/backends/kdeconnect/kdeconnectmanager.cpp, line 116 > > <https://git.reviewboard.kde.org/r/123263/diff/1/?file=360137#file360137line116> > > > > Answering your question from this patch's description: yes, I prefer > > that you call onDeviceAdded() slot for each device appended here. You can > > also use a watcher in this class' constructor to update m_devices > > asynchronously. Otherwise, no KdeConnectManager::deviceAdded() signal is > > emitted during KdeConnectManager startup even though m_devices is not > > empty. This method should just return the ids from m_devices. > > Kai Uwe Broulik wrote: > Ok, thanks, I wasn't sure how that internally works. > > So I populate the list async and emit a deviceAdded signal for each of > them no matter whether I found them on the first sweep or when they were > actually added afterwards.
Yes. - Lamarque ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123263/#review78531 ----------------------------------------------------------- On April 5, 2015, 3:57 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123263/ > ----------------------------------------------------------- > > (Updated April 5, 2015, 3:57 p.m.) > > > Review request for Solid, Albert Vaca Cintora and Emmanuel Pescosta. > > > Repository: solid > > > Description > ------- > > This adds a KDE Connect backend to Solid enabling us to show phone battery > status in the device notifier, and have the phone appear in the device > notifier like an external storage. > > It is pretty synchronous at the moment. Any ideas how I could improve that? I > guess when someone wants to get devices, it creates a new device interface > and calls allDevices() or devicesFromQuery() immediately afterwards, so I > cannot just do that stuff async and defer population? Or call deviceAdded for > each of the initial ones once I have them? > > It requires adjustment in the kdeconnect KIO slave so it can handle addresses > like kdeconnect://org/kde/kdeconnect/123456 or > kdeconnect:udi=/org/kde/kdeconnect/123456 (like mtp kio does) because all > Dolphin or the device notifier have is the UDI. It also needs a new device > action that matches "Portable Media Player" with the "kdeconnect" protocol, > similar to mtp. > > By adjusting the predicate in Dolphin's Places sidebar (and probably the file > dialog) to query for the "kdeconnect" protocol, we can have its places > automatically updated without having KDE Connect manually mess with > KFilePlaces. > > > Diffs > ----- > > src/solid/devices/CMakeLists.txt 9271ae1 > src/solid/devices/backends/kdeconnect/CMakeLists.txt PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectbattery.h PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectbattery.cpp PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectdevice.h PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectdevice.cpp PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectdeviceinterface.h > PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectdeviceinterface.cpp > PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectmanager.h PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectmanager.cpp PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectportablemediaplayer.h > PRE-CREATION > src/solid/devices/backends/kdeconnect/kdeconnectportablemediaplayer.cpp > PRE-CREATION > src/solid/devices/managerbase.cpp eee4de5 > > Diff: https://git.reviewboard.kde.org/r/123263/diff/ > > > Testing > ------- > > With some crash fixes already pushed to plasma-workspace it works pretty well. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel
