qt5 converting QCombBox setCurrentText to setCurrentIndex in qsstv

zentara · 08-02-2018, 05:35 AM

Hi, I am trying to compile a qt5 app for slow scan tv called qsstv. http://users.telenet.be/on4qz/qsstv/index.html.
I ran into an error where it says:

Code:

./ui_txwidget.h:834:33: error:  ‘class QComboBox’ has no member named ‘setCurrentText’; did you mean ‘setCurrentIndex’?
 drmTxBandwidthComboBox->setCurrentText(QApplication::translate("txWidget", "2.2 kHz", 0, QApplication::UnicodeUTF8));

I encountered 4 of these lines, and after just commenting them out, the app compiles fine. However, I would like to correct the error, because 2.2 kHz is probably an important initial setting, but am not familiar enough with qt, and was wondering if someone could give me the right syntax.

I tried changing setCurrentText to setCurrentIndex, but then got errors about the argument list being of the wrong type.

So, if you would please, what would be the correct syntax to change

Code:

          drmTxBandwidthComboBox->setCurrentText(QApplication::translate("txWidget", "2.2 kHz", 0, QApplication::UnicodeUTF8));

to use setCurrentIndex?

Thank You,
zentara

zentara · 08-02-2018, 07:58 AM

Hi, I hate to be so inept as to answer my own question, but I put on my qt sourceror's apprenctice cap and came up with the following fix. It compiles and seems to work, but I don't know if it is the best solution.

Code:

/* changed setCurrentText  to setItemText with a first arg of 0 */

drmTxBandwidthComboBox->setCurrentText( QApplication::translate("txWidget", "2.2 kHz", 0, QApplication::UnicodeUTF8));
to
drmTxBandwidthComboBox->setItemText(0, QApplication::translate("txWidget", "2.2 kHz", 0, QApplication::UnicodeUTF8));

Any comments or better solutions welcome.

dugan · 08-02-2018, 08:08 AM

I don't know why it's not working (I couldn't find any references in Qt's documentation to setCurrentText actually being removed), but it looks like you can fix it by changing the "text" property to the "index" property in the .ui files.

https://forum.qt.io/topic/45373/a-qu...currentindex/2
https://forum.qt.io/topic/5120/qcomb...tcurrenttext/3

zentara · 08-02-2018, 09:14 AM

Hi, I'm using Slackware current from Jul 2018 with qt 5.7.1. I looked thru the links you mentioned plus others, it seems that setCurrentText was deprecated way back in qt4.
http://www.qtcentre.org/threads/8515...-is-deprecated
What thru me off was the error message hinting that I probably should be using setCurrentIndex. setItemText seems to be the better suggestion. I never could figure out what they meant by changing the text and index properties in the .ui files. Thanks for the reply.

jonathan018 · 04-20-2019, 12:40 PM

Hello,
Just yesterday updated to qsstv-9.3.3 on slackware64-14.2. Except zentara's function change, also was needed -std=c++11 switch.

73 de YU1IS