D17122: RFC: Use wl_display_add_socket_auto by default

[Fabian Vogt]

fvogt created this revision.
fvogt added reviewers: KWin, Plasma.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
fvogt requested review of this revision.

REVISION SUMMARY
  If there is no socketName specified explicitly, it will use
  wl_display_add_socket_auto to allocate the next free socket.
  The resulting name can be retrieved using socketName after a successful
  start afterwards.
  
  This is a RFC for now because this changes the default value of socketName
  (and causes the respective autotest to fail).
  
  If that's ok, I'll adjust the test accordingly. If not, please suggest how this
  should be implemented, a new autoSocketName bool property?

TEST PLAN
  Ran the new autotest, passes. kwin_wayland picks the next free
  socket without modifications and WAYLAND_DISPLAY is set properly.

REPOSITORY
  R127 KWayland

BRANCH
  master

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

AFFECTED FILES
  autotests/server/test_display.cpp
  src/server/display.cpp
  src/server/display.h

To: fvogt, #kwin, #plasma
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181123/6a8a8183/attachment.html>