D16663: [import] Fix leaking AbstractFileFilters

[Alexander Semke]

asemke added inline comments.

INLINE COMMENTS

> croick wrote in ImportFileWidget.cpp:595
> The point of keeping the filter is not just a correct cleanup, but also to omit reparsing the file that is to be imported. `currentFileFilter()` gets called a lot and makes the UI unresponsive otherwise. (Which was my initial reason for this patch.)
> I could switch to a raw pointer still, but that would be inconsistent with the pointers for the options widgets.

Right, I overlooked this important part with omitting the reparsing of the file. But also this can be done with the raw pointers. Which inconsistency with the options widgets do you mean?

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/20181207/84e74636/attachment.html>