On May 7, 2015, 7:24 a.m., Alex Richardson wrote:
Seems like this is duplicated in a few places already so I agree we should
add it. But won't most users of the API want only a single plugin returned?
Maybe also add a function `KPluginMetaData
KPluginLoader::findPluginById(const
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123669/
---
(Updated June 6, 2015, 4:59 a.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123669/#review81254
---
Ship it!
Ship It!
- Marco Martin
On May 6, 2015, 11:21
On May 7, 2015, 8:24 a.m., Alex Richardson wrote:
Seems like this is duplicated in a few places already so I agree we should
add it. But won't most users of the API want only a single plugin returned?
Maybe also add a function `KPluginMetaData
KPluginLoader::findPluginById(const
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123669/#review80013
---
Seems like this is duplicated in a few places already so I
On May 7, 2015, 7:24 a.m., Alex Richardson wrote:
Seems like this is duplicated in a few places already so I agree we should
add it. But won't most users of the API want only a single plugin returned?
Maybe also add a function `KPluginMetaData
KPluginLoader::findPluginById(const
On May 6, 2015, 11:24 p.m., David Edmundson wrote:
autotests/kpluginloadertest.cpp, line 357
https://git.reviewboard.kde.org/r/123669/diff/1/?file=367268#file367268line357
not that it really matters, but invalid is spelt wrong.
Double the invalidity.
it says invalid id,
On May 7, 2015, 7:24 a.m., Alex Richardson wrote:
Seems like this is duplicated in a few places already so I agree we should
add it. But won't most users of the API want only a single plugin returned?
Maybe also add a function `KPluginMetaData
KPluginLoader::findPluginById(const
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123669/
---
Review request for KDE Frameworks, Alex Richardson and David Faure.
---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123669/#review80001
---
+1 seems sensible.
autotests/kpluginloadertest.cpp (line
10 matches
Mail list logo