D20859: Crash in sddmthemeinstaller invalid use of errorString

[David Edmundson]

davidedmundson added a comment.


  > What's the harm in adding a line of defensive code so that we conform to the API?
  
  First step to fixing a bug is understanding the bug.
  
  Your description of what is happening does not match the code.
  
  From KJob.
  
    int KJob::error() const
    {
        return d_func()->error;
    }
    
    QString KJob::errorText() const
    {
        return d_func()->errorText;
    }
  
  You can't have a state where one of those is callable and not the other.

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

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

To: johngehrig, ngraham, davidedmundson, #plasma
Cc: anthonyfieroni, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190429/bc2db3da/attachment.html>