<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/105173/">http://git.reviewboard.kde.org/r/105173/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 9th, 2012, 10:48 a.m., <b>Aaron J. Seigo</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">on possibility is to make some changes to QScriptValue ScriptEnv::print(QScriptContext *context, QScriptEngine *engine) in kde-runtime/plasma/scriptengines/javascript/common/scriptenv.cpp to do what we did in the desktop scripting: have print() also emit a signal that contains the message, connect that to a printMessage(const QString &) (or whatever) signal in the ScriptEngine plugins and then you could just connect to that signal.
that will only cover the JavaScript script engines, however.
for QML, this might be of interest: http://qt-project.org/wiki/QML_Script_Console ..
for python/ruby/etc. i'm not sure what to suggest. one more downside of supporting those languages in plasmate ... :/
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">What's wrong with the current solution?
This is a normal question, I am thinking why this solution isn't good but I can't find any reason...
So I will explain why I followed this solution.
--First we have to take the output from javascript engines.
The Qt people has mentioned that the QtScript module is done,(the next one is an assumption, correct me if I am wrong) so all those stuff we be left behind since in Qt5 we will use
QML2 and the QJS* stuff.(this is blocker 1)
--the we need the output from QML engines(this code can be either pure QML code or c++ code which is exported to QML)
We can take this output from QDeclarative::errors() but I believe that we can't.(this is blocker 2)
So if we go to another solution we have two blockes, now we have only the downside that we will have to use
something like plasmatedebug
Also a side note about the current implementation,
right now the konsole takes the exact same output that [1]plasmate gives from q/kDebug, console.log, and in general everything that uses
Qt's debug mechanism.
>for python/ruby/etc. i'm not sure what to suggest. one more downside of supporting those languages in plasmate ... :/
If python/ruby/etc.. uses Qt's debug mechanism I believe that the above will work.(I don't use python/ruby/etc so I can't test it)
Also as it regards plasmate and other languages like python,ruby,etc.
I believe that the people who contribute to plasmate that doesn't use the above languages so sooner or later plasmate's relative
stuff with those languages will become unusable.
[1] when plasmate loads a package like rssnow, plasmate via the plasma-previewer will also contain the output from that package
</pre>
<br />
<p>- Giorgos</p>
<br />
<p>On June 7th, 2012, 1:01 p.m., Giorgos Tsiapaliwkas wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Giorgos Tsiapaliwkas.</div>
<p style="color: grey;"><i>Updated June 7, 2012, 1:01 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hello,
the goal is to implement a konsole which will take the log
from the previewer.
Please check this wiki page http://www.developer.nokia.com/Community/Wiki/How_to_redirect_qDebug_output_to_a_file
this patch addes konsole previewer into plasmate.
KonsolePreviewer is a dockwidget.
Also It is able to take the log from plasmate's embedded previewer.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The bad thing is that it takes the entire log of plasmate but we only
want to take the output from the previewer.
So I was thinking to implement a plasmateDebug() << "text" macro/method which
will start with a unique string which I will check during customMessageHandler().</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(12f8a3a)</span></li>
<li>main.cpp <span style="color: grey">(1c5ea19)</span></li>
<li>mainwindow.h <span style="color: grey">(520c26c)</span></li>
<li>mainwindow.cpp <span style="color: grey">(6f72624)</span></li>
<li>previewer/plasmoid/plasmoidpreviewer.h <span style="color: grey">(9cad9e4)</span></li>
<li>previewer/plasmoid/plasmoidpreviewer.cpp <span style="color: grey">(29853db)</span></li>
<li>previewer/plasmoid/plasmoidview.cpp <span style="color: grey">(3d40d48)</span></li>
<li>previewer/plasmoid/previewcontainment.h <span style="color: grey">(fb10941)</span></li>
<li>previewer/plasmoid/previewcontainment.cpp <span style="color: grey">(42b6af7)</span></li>
<li>previewer/previewer.h <span style="color: grey">(2907598)</span></li>
<li>previewer/previewer.cpp <span style="color: grey">(c9d7479)</span></li>
<li>previewer/runner/runnerpreviewer.h <span style="color: grey">(054c58d)</span></li>
<li>previewer/runner/runnerpreviewer.cpp <span style="color: grey">(5c8dc6d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105173/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>