D16663: [import] Fix leaking AbstractFileFilters

Christoph Roick noreply at phabricator.kde.org
Tue Dec 4 23:57:50 GMT 2018


croick updated this revision to Diff 46877.
croick retitled this revision from "[import] Use QSharedPointer for file filters" to "[import] Fix leaking AbstractFileFilters".
croick edited the summary of this revision.
croick added a comment.


  - rename currentFilter to m_currentFilter and don't use QSharedPointer at all

REPOSITORY
  R262 LabPlot

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D16663?vs=46698&id=46877

BRANCH
  filterpointer

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

AFFECTED FILES
  src/backend/datasources/LiveDataSource.cpp
  src/backend/datasources/MQTTClient.cpp
  src/backend/datasources/MQTTClient.h
  src/backend/datasources/MQTTTopic.cpp
  src/backend/datasources/MQTTTopic.h
  src/backend/datasources/filters/AbstractFileFilter.h
  src/backend/datasources/filters/AsciiFilter.cpp
  src/backend/datasources/filters/AsciiFilter.h
  src/backend/datasources/filters/BinaryFilter.cpp
  src/backend/datasources/filters/FITSFilter.cpp
  src/backend/datasources/filters/HDF5Filter.cpp
  src/backend/datasources/filters/ImageFilter.cpp
  src/backend/datasources/filters/JsonFilter.cpp
  src/backend/datasources/filters/NetCDFFilter.cpp
  src/backend/datasources/filters/NgspiceRawAsciiFilter.cpp
  src/backend/datasources/filters/NgspiceRawBinaryFilter.cpp
  src/backend/datasources/filters/ROOTFilter.cpp
  src/kdefrontend/datasources/FITSOptionsWidget.cpp
  src/kdefrontend/datasources/HDF5OptionsWidget.cpp
  src/kdefrontend/datasources/ImportFileDialog.cpp
  src/kdefrontend/datasources/ImportFileWidget.cpp
  src/kdefrontend/datasources/ImportFileWidget.h
  src/kdefrontend/datasources/NetCDFOptionsWidget.cpp
  src/kdefrontend/datasources/ROOTOptionsWidget.cpp

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/20181204/1fbca77a/attachment.html>


More information about the kde-edu mailing list