Wireshark-dev: Re: [Wireshark-dev] MSVC gives warnings "qt_ui_utils.cpp(208, 25): warning C4996

From: Roland Knall <rknall@xxxxxxxxx>
Date: Fri, 11 Feb 2022 11:46:31 +0100
Which was used in the past, but that function has other issues (opening the users documents if the trace file path contains spaces)

I am looking into it. Empty StringList seems to solve it, but had not been able to test it properly yet

Cheers

Am 11.02.2022 um 11:35 schrieb Graham Bloice <graham.bloice@xxxxxxxxxxxxx>:


From the online Qt docs:

bool QProcess::startDetached(const QString &command)

This function is obsolete. It is provided to keep old source code working. We strongly advise against using it in new code.

Hence the MSVC warning, C4996 for a function that's marked as deprecated.  According to the Qt docs you should be using the more extensive overload:

bool QProcess::startDetached(const QString &program, const QStringList &arguments, const QString &workingDirectory = QString(), qint64 *pid = nullptr)

and you'll have to provide the arguments parameter, even if it's an empty QStringList.

On Fri, 11 Feb 2022 at 10:15, Anders Broman via Wireshark-dev <wireshark-dev@xxxxxxxxxxxxx> wrote:

Hi,

Latest released one

C:\Qt\5.15.2

Regards

Anders

 

From: Roland Knall <rknall@xxxxxxxxx>
Sent: den 11 februari 2022 11:12
To: Developer support list for Wireshark <wireshark-dev@xxxxxxxxxxxxx>
Cc: Anders Broman <anders.broman@xxxxxxxxxxxx>
Subject: Re: [Wireshark-dev] MSVC gives warnings "qt_ui_utils.cpp(208, 25): warning C4996: 'QProcess::startDetached'"

 

Which Qt version are you on?

 

Am Fr., 11. Feb. 2022 um 11:06 Uhr schrieb Anders Broman via Wireshark-dev <wireshark-dev@xxxxxxxxxxxxx>:

Hi,

Just built and got the following warning:

 

         ..\ui\qt\utils\qt_ui_utils.cpp(208,25): warning C4996: 'QProcess::startDetached': Use QProcess::startDetached(const QString &program, const QStringList &arguments) instead [C:\Development\wsbuild64\ui\qt\qtui.vcxproj]

       ..\ui\qt\utils\qt_ui_utils.cpp(208,25): warning C4996:     success = QProcess::startDetached(command); [C:\Development\wsbuild64\ui\qt\qtui.vcxproj]

       ..\ui\qt\utils\qt_ui_utils.cpp(208,25): warning C4996:                         ^ [C:\Development\wsbuild64\ui\qt\qtui.vcxproj]

 

    1 Warning(s)

 

Regards

Anders

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe


--
Graham Bloice