D8779: Avoid hypothetical crash if output->init fails

2017-11-15 Thread David Edmundson
This revision was automatically updated to reflect the committed changes. Closed by commit R108:6325749f2117: Avoid hypothetical crash if output-init fails (authored by davidedmundson). REPOSITORY R108 KWin CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8779?vs=22247=22399 REVISION

D8779: Avoid hypothetical crash if output->init fails

2017-11-13 Thread Martin Flöser
graesslin accepted this revision. graesslin added a comment. This revision is now accepted and ready to land. misclicked, wanted to accept it. REPOSITORY R108 KWin BRANCH master REVISION DETAIL https://phabricator.kde.org/D8779 To: davidedmundson, #plasma, graesslin Cc: bshah,

D8779: Avoid hypothetical crash if output->init fails

2017-11-13 Thread Bhushan Shah
bshah added a comment. @graesslin, you requested changes here but what change? There is no comment REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8779 To: davidedmundson, #plasma, graesslin Cc: bshah, plasma-devel, kwin, #kwin, bwowk, ZrenBot, progwolff, lesliezhai,

D8779: Avoid hypothetical crash if output->init fails

2017-11-13 Thread Martin Flöser
graesslin requested changes to this revision. This revision now requires changes to proceed. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D8779 To: davidedmundson, #plasma, graesslin Cc: plasma-devel, kwin, #kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed,

D8779: Avoid hypothetical crash if output->init fails

2017-11-12 Thread David Edmundson
davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: KWin. Restricted Application added subscribers: KWin, kwin, plasma-devel. REVISION SUMMARY con->setOutput and crtc->setOutput is called in Output::~Output so this is doing