D18982: prevent assertion in regex load

Dominik Haumann noreply at phabricator.kde.org
Wed Feb 13 15:24:06 GMT 2019


dhaumann added a comment.


  I think that makes a lot of sense. Maybe we should even always print the warning, not only in debug mode? See comment below...
  
  PS: When submitting patches without phabricator's arc tool, could you add more diff context, like `git diff -U99 ...`. Then reviewing is much more simple.

INLINE COMMENTS

> rule.cpp:579-582
> +    if (Log().isDebugEnabled()) {
> +        if (!m_regexp.isValid())
> +            qCDebug(Log) << "Invalid regexp:" << m_regexp.pattern();
> +    }

I even wonder whether this should be always printed, i.e.:

  const bool isValid = m_regexp.isValid();
  if (!isValid)
      qCWarning(...) << "Invalid regexp:" << m_regexp.pattern();
  }
  return isValid && !m_regexp.pattern().isEmpty();

REPOSITORY
  R216 Syntax Highlighting

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

To: davschul, vkrause
Cc: dhaumann, kwrite-devel, kde-frameworks-devel, gennad, michaelh, ngraham, bruns, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190213/b5279d8f/attachment.html>


More information about the Kde-frameworks-devel mailing list