D16663: [import] Fix leaking AbstractFileFilters

[Christoph Roick]

croick marked an inline comment as done.
croick added a comment.


  Thanks for looking into this!
  I think you have a point about consistency with raw pointers. If we make sure that any use of `setFilter(filter)` is followed by releasing ownership of the `filter`, then that should be fine.
  I will add a line in the function documentation to make it clear.
  
  Changes to MQTT I will submit in a separate commit and remove some whitespace-only changes that I just discovered.
  
  Would you be fine with this solution?

INLINE COMMENTS

> ImportFileWidget.cpp:106
> -	m_jsonOptionsWidget(nullptr),
> -	m_rootOptionsWidget(nullptr),
>  	m_fileName(fileName),

Will do this in a separate commit

> asemke wrote in ImportFileWidget.h:103
> m_currentFilter instead of currentFilter for the private member? Why mutable here?

currentFileFilter() is marked const, but m_currentFilter is changed therein.

REPOSITORY
  R262 LabPlot

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

To: croick, #labplot, asemke
Cc: asemke, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181205/4ada3377/attachment.html>