Good point
Op 30 apr. 2014 02:02 schreef "Clément Brisset" <notifications@github.com>:

> In interface/src/ui/ScriptEditorWidget.cpp:
>
> > +        QTextStream in(&file);
> > +        _scriptEditorWidgetUI->scriptEdit->setPlainText(in.readAll());
> > +        setScriptFile(scriptPath);
> > +
> > +        disconnect(this, SLOT(onScriptError(const QString&)));
> > +        disconnect(this, SLOT(onScriptPrint(const QString&)));
> > +    } else {
> > +        QNetworkAccessManager* networkManager = new QNetworkAccessManager(this);
> > +        QNetworkReply* reply = networkManager->get(QNetworkRequest(url));
> > +        qDebug() << "Downloading included script at" << scriptPath;
> > +        QEventLoop loop;
> > +        QObject::connect(reply, SIGNAL(finished()), &loop, SLOT(quit()));
> > +        loop.exec();
> > +        _scriptEditorWidgetUI->scriptEdit->setPlainText(reply->readAll());
> > +        if (!saveAs()) {
> > +            emit static_cast<ScriptEditorWindow*>(this->parent()->parent()->parent())->terminateCurrentTab();
>
> I don't quite get what you want to do here.
> Why not just call terminateCurrentTab() ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/highfidelity/hifi/pull/2748/files#r12123797>
> .
>