<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 10, 2019, 7:36 PM Friedrich W. H. Kossebau <<a href="mailto:kossebau@kde.org">kossebau@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Michael,<br>
<br>
Am Samstag, 9. März 2019, 16:53:58 CET schrieb Michael Reeves:<br>
>    I would like to move forward with a 1.8 release targeting end of Apirl.<br>
> Could someone please check over the kf5/qt5 changes to make sure there are<br>
> no major problems? Also there is custom painter that tries to handle left<br>
> to right text manually what is the proper way to support this? I will be<br>
> doing at least the next few releases independent of Applications do to the<br>
> amount time past since the last release for kdiff3. The 1.8 branch creation<br>
> and freeze is set for 3-22 if no major issues are found.<br>
<br>
When I gave it a (quick) look earlier today, nothing grave catched my eyes <br>
when it comes to Qt5/kF5 interfacing code (just a few favourite nitpicks of <br>
mine, which are already reviewed/pushed :) ).<br>
No idea about the custom RL painter myself or proper ways.<br>
<br>
Two things I noticed though which you ideally give some look:<br>
<br>
kdiff3 -h & -v show both a window as well as print output on the commandline.<br>
The windows are a bit unexpected and unusual given one already is on the <br>
commandline, perhaps that feature to also show a window could be removed?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div><div dir="auto">  isatty looks like it should be able to tell when the output is going to the terminal and disable the windows in that case. Not sure about if it's going to a file. I look into this a little more.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
You might also want to port the debug output from fprintf(stderr, ...) & Co. <br>
to qCDebug() & Co.<br></blockquote></div></div><div dir="auto">Definitely will do. I don't really like the direct system call use where a qt equivalent exists. Qt also provide s configure this easily without a rebuild.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers<br>
Friedrich<br>
<br>
<br>
</blockquote></div></div></div>