port libmyth to Qt4

[danielk]

Preliminary patch

[danielk]

The patch currently breaks a lot of things, and there are a lot of TODO's in the code, but it compiles :)

[Nigel]

(In [18164]) A few trivial QString fixes found by Daniel. See #5596

[danielk]

Updated to current svn

[danielk]

Adds a couple Qt3 UI classes to mythmusic to allow plugin to compile with qt4 libmyth.

[danielk]

Implements most of the TODO's in the patch

[danielk]

Note: As of the last patch all the code segments marked TODO in the earlier patch have been implemented. There are still problems remaining, mostly having to do with signal/slots having been renamed in Qt4.

[Nigel]

Had lots of errors like:

diseqcsettings.cpp:950: error: no matching function for call to 'MythListBox::setCurrentRow(int, QItemSelectionModel::SelectionFlag)'
/Volumes/MythBuild/.osx-packager/build/include/QtGui/qlistwidget.h:229: note: candidates are: void QListWidget::setCurrentRow(int)

Instead of changing all the code's setSelected(), am wondering if it might be easier to have a wrapper method MythListBox::setSelected(const int i, const bool flag) { setCurrentRow(i); (void)flag; };

[danielk]

no improvements, just updated to the latest svn

[danielk]

Nigel, that's my fault. I used a Qt 4.4 function, in Qt 4.0 through 4.3 there is no selection model parameter. I'll fix this in the next round.

[danielk]

(In [18259]) Refs #5596. Changes cerr redirects to VERBOSE macros in xmlparse. A number of these were using the unsafe QString->cstring conversions disabled when you turn off Qt3 compatibility mode.

[danielk]

Updated patch, this one should compile with Qt 4.3.

[Nigel]

For some reason:

patching file mythtv/libs/libmyth/mediamonitor-unix.cpp
Hunk #3 FAILED at 160.

Still testing, but this really is mammoth. I might start applying the little parts that I have tested, if that's OK?

[danielk]

Nigel, more than ok. :)

I'm super busy with work for the next few weeks, but the smaller this patch gets the easier it will be to apply. The media monitor code contains some fixes too which I was planning to factor out anyway (I wasn't able to test the media monitor changes until I got the existing media monitor code to work on my machine first...)

There is also a known bug with the exit from mythtv dialog (when configured to prompt on exit), other dialogs I tested appeared to work, but this is probably a generic failure of some sort. My first suspicion with these is that some Qt slot has been renamed, but I didn't notice any complaints about that when entering mythtv and trying to exit immediately, so this may be something like a changed Dialog result code, or something else entirely. I haven't had a chance to look at it myself other than to note the problem.

[Nigel]

(In [18300]) Qt4 port the first two methods. Loosely based on Daniel's patch. See #5596

[mythdev@…]

The last patch does not compile, because in the header it says:

 static const QString           kUDEV_FIFO;

And the cpp says:

  const char * MediaMonitorUnix::kUDEV_FIFO = "/tmp/mythtv_media";

So the patch would be:

Index: libs/libmyth/mediamonitor-unix.h
===================================================================
--- libs/libmyth/mediamonitor-unix.h    (revision 18300)
+++ libs/libmyth/mediamonitor-unix.h    (working copy)
@@ -23,7 +23,7 @@

   protected:
     int                          m_fifo;
-    static const QString         kUDEV_FIFO;
+    static const char *          kUDEV_FIFO;
 };

 #endif // MYTH_MEDIA_MONITOR_H

[Nigel]

(In [18301]) Build error located/patched by mythdev at telfort dot nl. Refs #5596.

[Nigel]

(In [18317]) A few more of Daniel's Qt3->Qt4 fixes (see #5596), tidy a few VERBOSE messages. Note that GetCDROMBlockDevices() seems to be broken for me - stream.atEnd() seems to always think the /proc/sys/dev/cdrom/info file is empty?

[Nigel]

(In [18319]) The last of Daniel's Qt3->Qt4 in this file. See #5596. I am unable to test the CheckDeviceNotifications??() changes, because my udev setup only sends "mount/umount" notifications, not "add/remove" ones. The code is nearly identical to that in CheckMountable??(), so it should be good. (hint, someone could create a CheckRemovable??)

[Nigel]

(In [18323]) A few more of Daniel's Qt4 fixes that I have tested. See #5596

[Nigel]

(In [18324]) More of Daniel's Qt4 fixes. See #5596. Note that killAudioLock is in the base class, so it doesn't seem to need initialising in the ALSA derived sub-class.

[Nigel]

(In [18325]) Pointless char*->QString->char* conversion, which is dangerous in Qt4. Looks like it has been that way since the original MMon patch [2696]. Much easier to change these instances, and pass char* around, than change all the method signatures, like in Daniel's patch. See #5596

[Nigel]

(In [18326]) Qt4 QMutex initialiser. See #5596

[Nigel]

(In [18327]) Fix QString -> char * conversions which are unsafe under Qt4. See #5596

[Nigel]

(In [18328]) A few more Qt3isms gone. See #5596. I hadn't realised that QObject::tr(QString) is now unsafe. Its one of the few Qt methods that uses a char * instead of a QString. TrollTech? should provide a QString one?

[Nigel]

(In [18329]) Until we fully remove qt3 support, add a define. See #5596

[Nigel]

(In [18330]) Some plugins still need Q3UrlOperator. See #5596

[Nigel]

(In [18331]) Add a header for Qt4 porting. See #5596

[Nigel]

(In [18332]) Add a header for Qt4 porting. See #5596

[Nigel]

(In [18333]) One too many changes in [18328]. See #5596

[Nigel]

(In [18334]) Qt4 safe strings, deprecate Qt3 compat QGridLayout methods. See #5596. That is nearly the end of the easy stuff! Time to update the patch

[Nigel]

v9 less Nigel's recent commits

[Nigel]

(In [18339]) Harmless header addition. (I missed this in [18331]). Refs #5596

[Nigel]

v10 plus a few omissions

[Nigel]

(In [18390]) Remove Qt3-isms. Slightly simpler version of Daniel's patch. See #5596. (I changed the MythListBox? to a QListWidget for now)

[Nigel]

(In [18391]) Another Qt4 QString safety thing. See #5596

[Nigel]

(In [18432]) Rationalise mythcontext.h usage, unwrap long lines, cerr -> VERBOSE. Inspired by Daniel's Qt4 patch. See #5596.

[Nigel]

v12 patch has some of the MythThemedDialog? subclasses that were missed (in the plugins). Looking at all those method signature changes, I think we should just stay with the char*, and do this in mythwidgets.cpp and mythdialogs.cpp.

    if (name)
        setObjectName(QString(name));

if necessary.
I'm hoping to have some time to change settings.cpp to use the new MythUI, but we'll see?

[Nigel]

(In [18473]) Header tidyup, Q3Frame -> QFrame, add some object names. See #5596. Also changes all VERBOSE -> cerr

[Nigel]

(In [18474]) Cut/paste/typo in 18473. See #5596

[Nigel]

(In [18475]) The other part of [18473] - one Q3VBox... to QBox... - see #5596

[Nigel]

(In [18478]) Lastly (fow a few hours), const-ify a few QString parameters. See #5596. Use consistent QString null initializers (I prefer = QString() - its shortest).

[Nigel]

(In [18483]) Pass const QString refs to MythThemedDialog?, default object name. See #5596

[Nigel]

(In [18516]) Default widget object names, lots of whitespace changes to reduce size of future patching. See #5596

[Nigel]

(In [18517]) Default widget object names, tidy Qt header names, some Qt3 compat removal. See #5596

[Nigel]

Patch against r18518

[danielk]

just updated patch to latest svn

[danielk]

(In [18571]) Refs #5596. Use Qt4 QRegExp::indexIn() instead of Qt3 only QRegExp::search()

[danielk]

(In [18585]) Refs #5596. Converts Qt3 only container classes in libmyth to Qt4 compatible containers. This changes the binary revision and so requires relinking the plugins.

[danielk]

updated patch

[danielk]

(In [18589]) Refs #5596. Port MythListBox? to Qt4.

Note: MythMusic was the only user of MythListView? and also used some funky aspects of MythListBox?, so it now has it's own copies of these two Qt3 classes and MythListView? is gone from libmyth. I didn't try to port mythmusic to Qt4 since I understand there is already an effort to port it to MythUI.

[danielk]

updated patch

[danielk]

(In [18595]) Refs #5596. Revert [18589]. This caused some serious layout problems in mythtv-setup, I think we need to commit the Qt3->Qt4 layout class conversion before committing this..

[danielk]

(In [18598]) Refs #5596. This applies a number of smaller Qt4 fixes to libmyth. Mostly methods that have been renamed or otherwise changed slightly since Qt3. Note: requires plugin relinking.

[danielk]

(In [18602]) Refs #5596. Reapply [18589]. The problem turned out to be an error in one line of code in the patch where an 'H' was used instead of a 'V'. This completely fubarred a number of layouts.

Note this does require relinking the plugins.. as will another patch later today.

[danielk]

(In [18603]) Refs #5596. Rest of [18602], committed from wrong directory.

[danielk]

Updated patch

[danielk]

(In [18605]) Refs #5596. Use Qt4 layout managers instead of Qt3 versions in libmyth. Note this does require relinking the plugins.

[danielk]

(In [18607]) Refs #5596. Switch from Q3EditText to QEditText. Requires relinking plugins.

[danielk]

Note: #5792 should be applied before closing this ticket, in addition to other changes, It changes some Qt3 compat calls to Qt4 calls.

[danielk]

Updated patch

[danielk]

(In [18608]) Refs #5596. Connect correct signal for selection updates in ProgLister?; this was broken by [18602] this morning.

[danielk]

(In [18609]) Refs #5596. Qt4.3 compatibility patch. This was in earlier versions of the patch in #5596, but fell through the cracks at some point.

[danielk]

(In [18610]) Refs #5596. Convert from Q3ButtonGroup to QButtonGroup in libmyth

[danielk]

what remains -- rich text widget removal

[danielk]

(In [18616]) Refs #5596. A number of small Qt3->Qt4 conversions. Note: plugins will need to be relinked.

[danielk]

(In [18622]) Refs #5596. A few of the remaining Qt3 constructs in libmyth. This changeset does _not_ require relinking plugins.

[danielk]

(In [18647]) Refs #5596. Port StackedConfigurationGroup? from Qt3 to Qt4.

[danielk]

(In [18649]) Refs #5596. Port MythWizard? to Qt4, remove TabbedConfiguratioGroup? (which wasn't being used anywhere).

[danielk]

(In [18650]) Refs #5596. Gets rid of Q3SimpleRichText.

Note the rendering is by no means as pretty with the new Qt4 rich text handler. It can be, but now we need to use CSS for the same effect, and I did not bother to make this pixel perfect since this should soon be replaced with a MythUI component.

[danielk]

(In [18652]) Refs #5596. More Qt4 fixes for MythWizard?.

[danielk]

(In [18655]) Fixes #5596. Removes last vestiges of Qt3 in libmyth

[Janne Grunau]

(In [18723]) fixes "Change Recording Group" broken due to incomplete qt4 porting in [18589]

The signal we were connecting to was removed from the Qt4 class. Refs #5596