Update LogLocale.cpp

[aa5sh]

On MacOS at least the time had a +0000 at the end not sure if that was intentional. It looked like you had a regex to try and remove some stuff at the end of the time format. I added another case to remove that.

[foldynl]

thanks. to be honest, every platform has it differently even if you use QT and what's worse is that every version of QT has it differently. I'll have to release this as a hotfix.

[foldynl]

Which version of Qt are you using?

[aa5sh]

That is what I was thinking since you had several versions. I am using 6.8.2 on my machine.

…

On Feb 16, 2025, at 4:25 AM, Ladislav ***@***.***> wrote: foldynl left a comment (foldynl/QLog#599) Which version of Qt are you using? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread. <#599 (comment)> <https://github.com/notifications/unsubscribe-auth/AUEEMXVEY3CZHY7L3ZATXUT2QBRRVAVCNFSM6AAAAABXGZYBXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRRGM3DIOJYGE> foldynl left a comment (foldynl/QLog#599) <#599 (comment)> Which version of Qt are you using? — Reply to this email directly, view it on GitHub <#599 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AUEEMXVEY3CZHY7L3ZATXUT2QBRRVAVCNFSM6AAAAABXGZYBXGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRRGM3DIOJYGE>. You are receiving this because you authored the thread.

[foldynl]

I think that we will do it better. I received another bug report with a different format, which is caused by Qt 6.8. Please, can you somehow run this piece of code (for example, add this to the constructor of the LogLocale class) and send me what it looks like on macOS? The output for Windows 6.8 is attached.

What's confusing is that your timeformat contains tt. That should no longer appear in the logformat in QT 6.8.

Locale_QT68_Windows.log

QSet<QString> uniqueFormats;

    QList<QLocale> locales = QLocale::matchingLocales(QLocale::AnyLanguage, QLocale::AnyScript, QLocale::AnyCountry);

    for (const QLocale &locale : locales) 
   {
        QString longFormat = locale.timeFormat(QLocale::LongFormat);
        if ( !uniqueFormats.contains(longFormat))
        {
            qInfo() << "  LongFormat:   " <<longFormat;
            uniqueFormats.insert(longFormat);
        }
    }

[aa5sh]

Locale_QT682_MacOS.log

Here you go.

[foldynl]

thank you. It seems that we will have to expand the regexp.

[foldynl]

just for completeness - flatpak QT 6.8.2

LocaleQT682_Flatpak.log

[aa5sh]

I combined the three log files and it looks like something like this grabs all of the t's and surrounding () and [] I see on some.
tttt|\(t\)|\bt\b|\btt\b|\(tttt\)|\[tttt\]

[foldynl]

Branch testing_0.42.1 contains the fix. Please, could you check it on Mac. Now I am checking Flatpak and Windows

[aa5sh]

Looks good in a quick check:

[foldynl]

thank you