dfaure created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  The QFileDialog documentation says these are equivalent:
  
    dialog.setNameFilter("All C++ files (*.cpp *.cc *.C *.cxx *.c++)");
    dialog.setNameFilter("*.cpp *.cc *.C *.cxx *.c++");
  
  so we should support both syntaxes.

TEST PLAN
  See autotest addition; also tested together with my kdialog port to 
QFileDialog.

BRANCH
  wildcard

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

AFFECTED FILES
  autotests/kfiledialog_unittest.cpp
  src/platformtheme/kdeplatformfiledialoghelper.cpp

To: dfaure, #plasma, apol
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart, lukas

Reply via email to