Fix startup segfault crash

Review Request #120721 - Created Oct. 22, 2014 and submitted

Information
Rahul Chowdhury
khangman
frameworks
170fd89...
Reviewers
kdeedu
whiting

Initially the app used to crash when started, and returned segmentation fault. In the KHangman class the show() function of KXmGuiWindow is overriden, which now checks for the kvtml files first. If no files are found, a QMessageBox is displayed, otherwise the mainwindow is loaded.


  

Issues

  • 4
  • 0
  • 0
  • 4
Description From Last Updated
This looks pretty good, I would remove the qDebug() lines though, they shouldn't be needed now that the problem is ... Jeremy Whiting Jeremy Whiting
No need to return; it will return outside the else block anyway. Jeremy Whiting Jeremy Whiting
this qDebug is still useful, but the other ones you've added should be removed Jeremy Whiting Jeremy Whiting
No need to leave commented out code, just remove it instead. Jeremy Whiting Jeremy Whiting
Jeremy Whiting
Rahul Chowdhury
Rahul Chowdhury
Jeremy Whiting
Rahul Chowdhury
Review request changed

Status: Closed (submitted)

Loading...