Merge lp:~josharenson/unity8/doc_args into lp:unity8

[Josh Arenson]

Implements usage-style documentation for unity8 executable.
Fixes lp:1269282

 * Are there any related MPs required for this MP to build/function as expected? Please list.
lp:~aacid/unity8/killqt51

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Compiled code, verified new feature works as expected, and ran ctest

 * Did you make sure that your branch does not contain spurious tags?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Dependent branch does. This MP does not

 * If you changed the UI, has there been a design review?
na

[Albert Astals Cid (aacid)]

Should increase the qt5 version number dependency in debian/control to 5.2 since QCommandLineParser is new for 5.2
Also probably in the find_package of CMakeListst.txt

review: Needs Fixing

[Albert Astals Cid (aacid)]

And you're missing https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 as part of your description of the change

[Albert Astals Cid (aacid)]

And finally, we don't support Qt translation system (we are using gettext) so for now i think we can just pretend we don't need to translate the commandline options and just drop the QCoreApplication::translate() and just pass the English text as second param of QCommandLineOption.

Saviq, what's your opinion?

[Michał Sawicz (saviq)]

Yeah, let's leave the help in English. It's not like they're general-use.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:858
http://jenkins.qa.ubuntu.com/job/unity8-ci/2862/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/391/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1726/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1383/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1387/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1383/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4628/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2862/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

In case you're confused by all the failings → whitespace issues

review: Needs Fixing

[Michał Sawicz (saviq)]

src/main.cpp: bad whitespace in lines 67, 84

[Albert Astals Cid (aacid)]

you probably want to merge lp:~aacid/unity8/killqt51 into here (you'll have to resubmit the MR setting a prerequisite branch)

[Josh Arenson (josharenson)]

> you probably want to merge lp:~aacid/unity8/killqt51 into here (you'll have
> to resubmit the MR setting a prerequisite branch)

So if I clean up my white space, depend on _your_ MP, and resubmit _this_ MP in the proper format, we should be good to go?

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:859
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~josharenson/unity8/doc_args/+merge/217158/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/2867/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/429
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1731
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1388
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1392
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1392/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1388
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/386
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4675
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4675/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6395

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2867/rebuild

review: Needs Fixing (continuous-integration)

[Michał Sawicz (saviq)]

You didn't merge Albert's branch into this one, and you didn't put it in the "prerequisite" field when resubmitting.

[Michał Sawicz (saviq)]

I've resubmitted this for you, you can merge Albert's branch, but that's not necessary, since the other one will be merged before anyway.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:859
http://jenkins.qa.ubuntu.com/job/unity8-ci/2868/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/434
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1732
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1389
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1393
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1393/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1389
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/391
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4681
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4681/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6402

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2868/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:860
http://jenkins.qa.ubuntu.com/job/unity8-ci/2872/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/451
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1736
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1393
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1397
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1397/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1393
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/403
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4704
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4704/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6423

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2872/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:860
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~josharenson/unity8/doc_args/+merge/217282/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/2873/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/453
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1737
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1394
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1398
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1398/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1394
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/404
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4706
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4706/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6424

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2873/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

This breaks the -windowgeometry param, i.e.

./builddir/src/unity8 -windowgeometry 400x400

Doesn't work anymore

review: Needs Fixing

[Josh Arenson (josharenson)]

> This breaks the -windowgeometry param, i.e.
>
> ./builddir/src/unity8 -windowgeometry 400x400
>
> Doesn't work anymore
Pushed a small hack to WAR this. Let me know if you have an idea for something more elegant.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:861
http://jenkins.qa.ubuntu.com/job/unity8-ci/2877/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/479
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1741
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1398
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1402
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1402/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1398
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/426
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4740
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4740/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6480

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2877/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

> > This breaks the -windowgeometry param, i.e.
> >
> > ./builddir/src/unity8 -windowgeometry 400x400
> >
> > Doesn't work anymore
> Pushed a small hack to WAR this. Let me know if you have an idea for something
> more elegant.

I think i'd prefer if you move the parsing of the argument to main.cpp and only pass to ApplicationArguments the already parsed size instead of the list of args

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:862
http://jenkins.qa.ubuntu.com/job/unity8-ci/2882/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/498
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1746
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1403
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1407
        deb: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1407/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1403
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/440
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4762
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4762/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6514

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2882/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:862
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~josharenson/unity8/doc_args/+merge/217840/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/2892/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-trusty-touch/542/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1756/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-amd64-ci/1413/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-armhf-ci/1417/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-trusty-i386-ci/1413/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/4817/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2892/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

I'm confused about the auto_ptr use

Wouldn't

std::auto_ptr<ApplicationArguments> appArgs;
if (parser.isSet(windowGeometryOption) &&
  parser.value(windowGeometryOption).split('x').size() == 2) {
  QStringList geom = parser.value(windowGeometryOption).split('x');
  appArgs.reset(new ApplicationArguments(geom.at(0).toInt(), geom.at(1).toInt()));
} else {
  appArgs.reset(new ApplicationArguments());
}
ApplicationArguments& qmlArgs = *appArgs;

make more sense as

ApplicationArguments qmlArgs;
if (parser.isSet(windowGeometryOption) &&
  parser.value(windowGeometryOption).split('x').size() == 2) {
  QStringList geom = parser.value(windowGeometryOption).split('x');
  qmlArgs.setSize(new ApplicationArguments(geom.at(0).toInt(), geom.at(1).toInt()));
}

or if you don't want to expose setSize (which is indeed fine since if you don't make it a q_invokable it won't be available from the qml side)

QSize size;
if (parser.isSet(windowGeometryOption) &&
  parser.value(windowGeometryOption).split('x').size() == 2) {
  QStringList geom = parser.value(windowGeometryOption).split('x');
  size = QSize(new ApplicationArguments(geom.at(0).toInt(), geom.at(1).toInt()));
}
ApplicationArguments qmlArgs(size);

?

Any of those two versions seem easier to read to me

// Note code not warranteed to compile :D

review: Needs Information

[lp:~josharenson/unity8/doc_args]

863. By Josh Arenson on 2014-05-05

Refactored ApplicationArguments constructor(s)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:863
http://jenkins.qa.ubuntu.com/job/unity8-ci/2910/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/9
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-trusty/1774/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-amd64-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/4
        deb: http://jenkins.qa.ubuntu.com/job/unity8-utopic-armhf-ci/4/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-utopic-i386-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/526
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/72
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/72/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/6795

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/2910/rebuild

review: Needs Fixing (continuous-integration)

[Michał Sawicz (saviq)]

Please strip tags: http://people.canonical.com/~msawicz/unity8/strip-u8-tags.sh

Do the same on your local checkout(s), too!

review: Needs Fixing

[Albert Astals Cid (aacid)]

Code looks good to me now, please ping us once the tags are cleared so we can top approve

* Did you perform an exploratory manual test run of the code change and any related functionality?
Yes, works

* Did CI run pass?
No, qmluitests failed to even start for some reason, but qmluitests don't cover these changes anyway

review: Approve

[Josh Arenson (josharenson)]

> Please strip tags:
> http://people.canonical.com/~msawicz/unity8/strip-u8-tags.sh
>
> Do the same on your local checkout(s), too!

Done

[Michał Sawicz (saviq)]

review: Abstain

=== modified file 'src/ApplicationArguments.h'
--- src/ApplicationArguments.h	2014-03-19 15:29:35 +0000
+++ src/ApplicationArguments.h	2014-05-05 22:46:16 +0000
@@ -28,14 +28,10 @@
 {
     Q_OBJECT
 public:
-    ApplicationArguments(const QStringList& args) {
-        if (args.contains(QLatin1String("-windowgeometry")) && args.size() > args.indexOf(QLatin1String("-windowgeometry")) + 1) {
-            QStringList geometryArg = args.at(args.indexOf(QLatin1String("-windowgeometry")) + 1).split('x');
-            if (geometryArg.size() == 2) {
-                m_size.rwidth() = geometryArg.at(0).toInt();
-                m_size.rheight() = geometryArg.at(1).toInt();
-            }
-        }
+    // Not exposed to the app as setSize isn't invokable
+    void setSize(int width, int height) {
+        m_size.rwidth()  = width;
+        m_size.rheight() = height;
     }
 
     Q_INVOKABLE bool hasGeometry() const { return m_size.isValid(); }

=== modified file 'src/main.cpp'
--- src/main.cpp	2014-04-02 11:19:54 +0000
+++ src/main.cpp	2014-05-05 22:46:16 +0000
@@ -18,6 +18,7 @@
  */
 
 // Qt
+#include <QCommandLineParser>
 #include <QtQuick/QQuickView>
 #include <QtGui/QGuiApplication>
 #include <QtQml/QQmlEngine>
@@ -43,6 +44,32 @@
 
     QGuiApplication::setApplicationName("Unity 8");
     QGuiApplication *application;
+
+    QCommandLineParser parser;
+    parser.setApplicationDescription("Description: Unity 8 Shell");
+    parser.addHelpOption();
+
+    QCommandLineOption fullscreenOption("fullscreen",
+        "Run in fullscreen");
+    parser.addOption(fullscreenOption);
+
+    QCommandLineOption framelessOption("frameless",
+        "Run without window borders");
+    parser.addOption(framelessOption);
+
+    QCommandLineOption mousetouchOption("mousetouch",
+        "Allow the mouse to provide touch input");
+    parser.addOption(mousetouchOption);
+
+    QCommandLineOption windowGeometryOption(QStringList() << "windowgeometry",
+            "Specify the window geometry as [<width>x<height>]", "windowgeometry", "1");
+    parser.addOption(windowGeometryOption);
+
+    QCommandLineOption testabilityOption("testability",
+        "DISCOURAGED: Please set QT_LOAD_TESTABILITY instead. \n \
+Load the testability driver");
+    parser.addOption(testabilityOption);
+
     if (isUbuntuMirServer) {
         QLibrary unityMir("unity-mir", 1);
         unityMir.load();
@@ -59,6 +86,11 @@
         application = new QGuiApplication(argc, (char**)argv);
     }
 
+    // Treat args with single dashes the same as arguments with two dashes
+    // Ex: -fullscreen == --fullscreen
+    parser.setSingleDashWordOptionMode(QCommandLineParser::ParseAsLongOptions);
+    parser.process(*application);
+
     QString indicatorProfile = qgetenv("UNITY_INDICATOR_PROFILE");
     if (indicatorProfile.isEmpty()) {
         indicatorProfile = "phone";
@@ -66,12 +98,17 @@
 
     resolveIconTheme();
 
-    QStringList args = application->arguments();
-    ApplicationArguments qmlArgs(args);
+    ApplicationArguments qmlArgs;
+    if (parser.isSet(windowGeometryOption) &&
+        parser.value(windowGeometryOption).split('x').size() == 2)
+    {
+      QStringList geom = parser.value(windowGeometryOption).split('x');
+      qmlArgs.setSize(geom.at(0).toInt(), geom.at(1).toInt());
+    }
 
     // The testability driver is only loaded by QApplication but not by QGuiApplication.
     // However, QApplication depends on QWidget which would add some unneeded overhead => Let's load the testability driver on our own.
-    if (args.contains(QLatin1String("-testability")) || getenv("QT_LOAD_TESTABILITY")) {
+    if (parser.isSet(testabilityOption) || getenv("QT_LOAD_TESTABILITY")) {
         QLibrary testLib(QLatin1String("qttestability"));
         if (testLib.load()) {
             typedef void (*TasInitialize)(void);
@@ -94,14 +131,14 @@
     view->engine()->setBaseUrl(QUrl::fromLocalFile(::qmlDirectory()));
     view->rootContext()->setContextProperty("applicationArguments", &qmlArgs);
     view->rootContext()->setContextProperty("indicatorProfile", indicatorProfile);
-    if (args.contains(QLatin1String("-frameless"))) {
+    if (parser.isSet(framelessOption)) {
         view->setFlags(Qt::FramelessWindowHint);
     }
 
     // You will need this if you want to interact with touch-only components using a mouse
     // Needed only when manually testing on a desktop.
     MouseTouchAdaptor *mouseTouchAdaptor = 0;
-    if (args.contains(QLatin1String("-mousetouch"))) {
+    if (parser.isSet(mousetouchOption)) {
         mouseTouchAdaptor = new MouseTouchAdaptor;
         application->installNativeEventFilter(mouseTouchAdaptor);
     }
@@ -125,7 +162,7 @@
     view->setSource(source);
     view->setColor("transparent");
 
-    if (qgetenv("QT_QPA_PLATFORM") == "ubuntu" || isUbuntuMirServer || args.contains(QLatin1String("-fullscreen"))) {
+    if (qgetenv("QT_QPA_PLATFORM") == "ubuntu" || isUbuntuMirServer || parser.isSet(fullscreenOption)) {
         view->showFullScreen();
     } else {
         view->show();