D21212: Remove DBus from Python backend

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

D21212: Remove DBus from Python backend

Malte Kraus
View Revision
sirgienko added inline comments.

View Inlineasemke wrote in pythonexpression.cpp:133

what happens if we have two images from two expressions and they arrive at the wrong order (second image is faster than the first one)?

All will worked, because each expressions have own m_tempFile.

View Inlineasemke wrote in pythonservermain.cpp:24

Are QTextStream and QChar really required in this file?

QTextStream no needed, but QChar used in recordSep, unitSep. Though, maybe i could change it to char.

View Inlineasemke wrote in pythonsession.cpp:141

use c++11 range based loop instead of the foreach macro.


View Inlineasemke wrote in pythonvariablemodel.cpp:49

is this int variable still required in the new code?

You right, the variable not needed, I just had forggoten to remove it.

R55 Cantor

To: sirgienko, asemke
Cc: kde-edu, asemke, narvaez, apol