T7994: Extract Qt Pulseaudio bindings into a Framework

Nicolas Fella noreply at phabricator.kde.org
Wed Jul 11 12:15:18 BST 2018


nicolasfella added a comment.


  In T7994#147341 <https://phabricator.kde.org/T7994#147341>, @sitter wrote:
  
  > Random complaints I stumbled upon:
  >
  > - `m_context = pa_context_new(api, "QPulse");` this should decidedly not hardcode `QPulse` but use https://doc.qt.io/qtthe-5/qcoreapplication.html#applicationName-prop (or maybe even displayName iff qapp can be cast to qguiapplication)
  > - along the same notion, Context should probably have a function `setName` for `pa_context_set_name` in case the qapp name isn't the desired name (e.g. with a plasmoid the qapp is plasma, but the app name should probably be the individual plasmoid)
  
  
  Seems reasonable, but I guess we can do it without breaking ABI.
  
  What would be the next step for an initial release?

TASK DETAIL
  https://phabricator.kde.org/T7994

To: nicolasfella
Cc: davidedmundson, michaelh, akrutzler, apol, sitter, drosca, #kde_connect, nicolasfella, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, ragreen, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, ZrenBot, seebauer, ngraham, bruns, bugzy, MayeulC, menasshock, lesliezhai, ali-mohamed, jensreuterberg, ach, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180711/4bd46aa3/attachment.html>


More information about the KDEConnect mailing list