D14050: Fwupd Backend For Review and Improvement

Abhijeet sharma noreply at phabricator.kde.org
Sun Jul 15 04:16:50 BST 2018


abhijeet2096 marked 18 inline comments as done.
abhijeet2096 added a comment.


  Still, Need to implement Notification Class and some functions in Updater Class. I have to see if we could use the standard Updater Class, after Confirmation I will use directly FwupdResource instead of abstract resource

INLINE COMMENTS

> apol wrote in SourcesPage.qml:113
> Remove unrelated patch

Actually, I modified this file to show EULA on setting page.

> anthonyfieroni wrote in FwupdBackend.cpp:85
> It's memory leak.

Please see the new modified function. Will Memory be leaking?

> apol wrote in FwupdBackend.cpp:100
> No need to cast. Use the FwupdResource directly.

Still Need To Fix it

> apol wrote in FwupdNotifier.cpp:37
> Don't add one that does nothing.
> Either implement it or just don't add it.

Need to Implement it, Like Updates which require Restart. Will Implement it.

> anthonyfieroni wrote in FwupdSourcesBackend.cpp:75-77
> ?

Here, The UI gets Updates first. The CheckBox fills first then its applied at the backend. Need a way to control the checkbox UI

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D14050

To: abhijeet2096, apol, davidedmundson
Cc: zzag, anthonyfieroni, plasma-devel, ragreen, ixoos, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180715/5fa2e6db/attachment-0001.html>


More information about the Plasma-devel mailing list