wbauer (Wolfgang Bauer)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Aug 30 2016, 3:41 PM (163 w, 4 d)
Availability
Available

Recent Activity

Fri, Oct 4

wbauer added a comment to D24413: Fix Increase/Decrease Volume shortcuts.

Since the method is only one line long, why not using lambda instead here?

Fri, Oct 4, 3:15 PM
wbauer added a comment to D24413: Fix Increase/Decrease Volume shortcuts.

PS: I think the argument is 0 because QAction::triggered() has a bool checked argument.
But Qt's new signal/slots syntax doesn't support default arguments anyway, according to the docs.

Fri, Oct 4, 1:22 PM
wbauer requested review of D24413: Fix Increase/Decrease Volume shortcuts.
Fri, Oct 4, 1:21 PM
wbauer committed R181:5ed62f9a089d: Register Play/Pause as global shortcut again (authored by wbauer).
Register Play/Pause as global shortcut again
Fri, Oct 4, 12:35 PM
wbauer closed D24381: Register Play/Pause as global shortcut again.
Fri, Oct 4, 12:35 PM
wbauer added a comment to D24381: Register Play/Pause as global shortcut again.

Btw, I just found a bug report about this:
https://bugs.kde.org/show_bug.cgi?id=373590

Fri, Oct 4, 12:33 PM

Thu, Oct 3

wbauer retitled D24381: Register Play/Pause as global shortcut again from Register Play/Pause global shortcut again to Register Play/Pause as global shortcut again.
Thu, Oct 3, 9:29 AM
wbauer updated the test plan for D24381: Register Play/Pause as global shortcut again.
Thu, Oct 3, 9:13 AM
wbauer updated the test plan for D24381: Register Play/Pause as global shortcut again.
Thu, Oct 3, 9:13 AM
wbauer updated subscribers of D24135: Fix missing directory separators when saving podcasts to disk.
Thu, Oct 3, 9:07 AM
wbauer added a comment to D24135: Fix missing directory separators when saving podcasts to disk.

+1 from me now as well.

Thu, Oct 3, 9:06 AM
wbauer updated the summary of D24381: Register Play/Pause as global shortcut again.
Thu, Oct 3, 9:03 AM
wbauer requested review of D24381: Register Play/Pause as global shortcut again.
Thu, Oct 3, 9:02 AM

Mon, Sep 30

wbauer accepted D24314: Improve KFloppy icon.

Thanks!

Mon, Sep 30, 6:36 PM · Frameworks
wbauer added a comment to D22258: Improve plugin caching.

I don't know the situation well enough to know whether it's valid that it's invalid or not. Let's follow that up in your bug report.

The problem apparently is that the mentioned plugins have their metadata in a .desktop file, but the code here creates the KFileMetaData from the .so file (so the metadata is empty).
See the bug report for more details.

Mon, Sep 30, 10:43 AM · Frameworks
wbauer added inline comments to D22258: Improve plugin caching.
Mon, Sep 30, 8:45 AM · Frameworks
wbauer added inline comments to D22258: Improve plugin caching.
Mon, Sep 30, 8:26 AM · Frameworks

Sun, Sep 22

wbauer added a comment to D24135: Fix missing directory separators when saving podcasts to disk.

You should add '/' instead of QDir::separator()...

Sun, Sep 22, 7:09 AM

Fri, Sep 20

wbauer closed D24040: Don't delete whole folder when deleting a track.
Fri, Sep 20, 9:12 AM
wbauer committed R181:b986f52d1d53: Don't delete whole folder when deleting a track (authored by wbauer).
Don't delete whole folder when deleting a track
Fri, Sep 20, 9:12 AM

Sep 19 2019

wbauer added a comment to D24040: Don't delete whole folder when deleting a track.

Looks OK. What is the status of the unit tests, though?

Sep 19 2019, 9:23 AM
wbauer committed R181:decd2a3ecb31: [TagDialog] Really enable "Open in filemanager" button for local files (authored by wbauer).
[TagDialog] Really enable "Open in filemanager" button for local files
Sep 19 2019, 8:31 AM
wbauer closed D24043: [TagDialog] Really enable "Open in filemanager" button for local files.
Sep 19 2019, 8:31 AM

Sep 18 2019

wbauer updated subscribers of D24043: [TagDialog] Really enable "Open in filemanager" button for local files.
Sep 18 2019, 8:54 AM
wbauer updated subscribers of D24040: Don't delete whole folder when deleting a track.
Sep 18 2019, 8:54 AM
wbauer updated the test plan for D24043: [TagDialog] Really enable "Open in filemanager" button for local files.
Sep 18 2019, 8:22 AM
wbauer requested review of D24043: [TagDialog] Really enable "Open in filemanager" button for local files.
Sep 18 2019, 8:17 AM
wbauer updated the test plan for D24040: Don't delete whole folder when deleting a track.
Sep 18 2019, 7:04 AM
wbauer requested review of D24040: Don't delete whole folder when deleting a track.
Sep 18 2019, 6:53 AM

Sep 10 2019

wbauer committed R443:6de7f7a49a3c: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Sep 10 2019, 2:18 PM
wbauer committed R443:5a1fe1578083: Fix build with cmake < 3.12 (authored by wbauer).
Fix build with cmake < 3.12
Sep 10 2019, 2:12 PM
wbauer committed R443:4e509b8735bf: Fix build with cmake < 3.12 (authored by wbauer).
Fix build with cmake < 3.12
Sep 10 2019, 2:09 PM
wbauer closed D23826: Fix build with cmake < 3.12.
Sep 10 2019, 2:09 PM · KDE Applications
wbauer requested review of D23826: Fix build with cmake < 3.12.
Sep 10 2019, 10:19 AM · KDE Applications

Sep 4 2019

wbauer added a comment to D23542: Promote KCM to top level and rename accordingly.

Please note that k3b ships the 'CDDB retrieval' KCM, which is in the Multimedia section as well

That's actually part of libkcddb and not k3b, btw.

Sep 4 2019, 5:49 AM · Plasma

Sep 1 2019

wbauer committed R73:a698894fdaac: Fix build with cmake < 3.7 (authored by wbauer).
Fix build with cmake < 3.7
Sep 1 2019, 6:52 PM
wbauer closed D23638: Fix build with cmake < 3.7.
Sep 1 2019, 6:52 PM · Rocs, KDE Edu
wbauer updated the summary of D23638: Fix build with cmake < 3.7.
Sep 1 2019, 12:17 PM · Rocs, KDE Edu
wbauer requested review of D23638: Fix build with cmake < 3.7.
Sep 1 2019, 11:45 AM · Rocs, KDE Edu
wbauer added a reviewer for D23635: Fix build with cmake < 3.7: Umbrello.
Sep 1 2019, 11:27 AM · Umbrello
wbauer requested review of D23635: Fix build with cmake < 3.7.
Sep 1 2019, 11:27 AM · Umbrello

Aug 29 2019

wbauer accepted D23570: FcSolveSolver: cleanup ressources.

Yes, this does fix the crash as well, and it builds fine with Qt 5.9.

Aug 29 2019, 6:15 PM
wbauer added a comment to D23570: FcSolveSolver: cleanup ressources.

This is hopefully already a minimal fix for the crash issues, and it does not depend on Qt 5.12 features. @wbauer It would be most helpful if you could test whether the original bug is indeed no longer occurring after this patch.

JFYI, I already tested the QScopeGuard fix from D23404 today, it alone does indeed fix the crash of BUG 395624.

Aug 29 2019, 5:08 PM
wbauer added inline comments to D23404: Revamp solver memory management.
Aug 29 2019, 8:11 AM
wbauer added a comment to D23404: Revamp solver memory management.

There is a problem though: QScopeGuard was introduced in Qt 5.12, while the minimum required version is 5.9.

Aug 29 2019, 8:09 AM
wbauer added a comment to D23404: Revamp solver memory management.

The issue with the previous version was that the memory accounting code was wrong, and the allocation would fail at some point. The most recent version fixes this, and I could successfully complete a game of Grandfather's Clock again.

Aug 29 2019, 8:03 AM

Aug 28 2019

wbauer added a comment to D23404: Revamp solver memory management.

Here's a valgrind log for now, maybe it contains a clue as well...

Aug 28 2019, 3:52 PM
wbauer added a comment to D23404: Revamp solver memory management.

Thank you very much for testing the patch. Could you maybe compile kpat with
cmake -DCMAKE_BUILD_TYPE=Debug -DECM_ENABLE_SANITIZERS='address,leak,undefined' ..
and report back any backtrace you get from playing Grandfather's Clock? I again cannot reproduce the crash :-(

Aug 28 2019, 3:45 PM
wbauer added a comment to D23404: Revamp solver memory management.

I did get another crash while playing Grandfather's Clock now though, no idea if it is related to this change or not.

I played 3 times and got the same crash every time (at some point during play), so it seems to be rather reproducible.

Aug 28 2019, 12:36 PM
wbauer added a comment to D23404: Revamp solver memory management.

I gave it a try and it does seem to fix the crashes from BUG 395624 here.

Aug 28 2019, 11:44 AM

Aug 8 2019

wbauer committed R433:838bebfd2fde: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 8 2019, 6:33 PM
wbauer committed R433:2ae63bea674c: [mirrorsearch] Remove no longer available services from default list (authored by wbauer).
[mirrorsearch] Remove no longer available services from default list
Aug 8 2019, 6:32 PM

Aug 7 2019

wbauer committed R36:84694592ceb5: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 7 2019, 9:22 PM
wbauer committed R36:c81ef41ac6a6: Fix build with cmake < 3.7 (authored by wbauer).
Fix build with cmake < 3.7
Aug 7 2019, 9:21 PM
wbauer closed D22983: Fix build with cmake < 3.7.
Aug 7 2019, 9:21 PM · Ark
wbauer committed R443:2e8696c1133c: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 7 2019, 6:51 PM
wbauer closed D22987: Fix build with cmake < 3.7.
Aug 7 2019, 6:51 PM · KDE Applications
wbauer committed R443:626358ec096a: Fix build with cmake < 3.7 (authored by wbauer).
Fix build with cmake < 3.7
Aug 7 2019, 6:51 PM
wbauer committed R226:3e2dc5be72f4: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 7 2019, 6:45 PM
wbauer committed R226:b48855158093: [webenginepart] Fix build with cmake < 3.7 (authored by wbauer).
[webenginepart] Fix build with cmake < 3.7
Aug 7 2019, 6:43 PM
wbauer closed D23006: [webenginepart] Fix build with cmake < 3.7.
Aug 7 2019, 6:43 PM · Konqueror
wbauer updated subscribers of D23006: [webenginepart] Fix build with cmake < 3.7.
Aug 7 2019, 4:56 PM · Konqueror
wbauer requested review of D23006: [webenginepart] Fix build with cmake < 3.7.
Aug 7 2019, 4:35 PM · Konqueror
wbauer committed R336:1df7a2a45070: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 7 2019, 4:16 PM
wbauer committed R336:36c658aaec85: Fix build with older cmake (authored by wbauer).
Fix build with older cmake
Aug 7 2019, 4:16 PM
wbauer closed D22990: Fix build with older cmake.
Aug 7 2019, 4:16 PM · KDE Edu, KDE Applications
wbauer committed R318:e39b7e8f5a91: Merge branch 'Applications/19.08' (authored by wbauer).
Merge branch 'Applications/19.08'
Aug 7 2019, 4:12 PM
wbauer committed R318:68aee5ed51b5: Fix build with cmake < 3.7 (authored by wbauer).
Fix build with cmake < 3.7
Aug 7 2019, 4:10 PM
wbauer closed D22984: Fix build with cmake < 3.7.
Aug 7 2019, 4:10 PM · Dolphin
wbauer requested review of D22990: Fix build with older cmake.
Aug 7 2019, 12:46 PM · KDE Edu, KDE Applications
wbauer requested review of D22987: Fix build with cmake < 3.7.
Aug 7 2019, 10:52 AM · KDE Applications
wbauer updated the diff for D22984: Fix build with cmake < 3.7.

Use NOT VERSION_LESS instead to minimize the patch.

Aug 7 2019, 9:57 AM · Dolphin
wbauer updated the diff for D22983: Fix build with cmake < 3.7.

Use NOT VERSION_LESS instead to minimize patch.

Aug 7 2019, 9:51 AM · Ark
wbauer added inline comments to D22983: Fix build with cmake < 3.7.
Aug 7 2019, 9:46 AM · Ark
wbauer requested review of D22984: Fix build with cmake < 3.7.
Aug 7 2019, 9:45 AM · Dolphin
wbauer added a reviewer for D22983: Fix build with cmake < 3.7: Ark.
Aug 7 2019, 9:29 AM · Ark
wbauer requested review of D22983: Fix build with cmake < 3.7.
Aug 7 2019, 9:29 AM · Ark
wbauer added a comment to D22814: Remove colon (:) prefix when looking up dictionary word.

Thank you!

Aug 7 2019, 8:47 AM · Plasma
wbauer added a comment to D22814: Remove colon (:) prefix when looking up dictionary word.

Can this be pushed to the 5.12 branch as well, please?
It's broken in 5.12.8 as well (https://bugs.kde.org/390776 was actually reported against 5.12.1), the patch applies cleanly and does fix the problem.

Aug 7 2019, 5:05 AM · Plasma

Jul 20 2019

wbauer added a comment to D22525: kioclient: Don't convert `:x:y` to `?line=x&column=y` for URLs starting with remote schemes..

What about the stable 5.16 branch?
It has only be committed to master so far AFAICS, so the fix would only end up in Plasma 5.17 which is still 3 months away...

Jul 20 2019, 5:29 PM · Plasma

Jul 12 2019

wbauer committed R220:8b039ceb4f6e: Merge branch '2.1' (authored by wbauer).
Merge branch '2.1'
Jul 12 2019, 2:16 PM
wbauer closed D22415: Actually set minimum Qt version.
Jul 12 2019, 2:14 PM
wbauer committed R220:fc49e3ec5d80: Actually set minimum Qt version (authored by wbauer).
Actually set minimum Qt version
Jul 12 2019, 2:14 PM
wbauer closed D22412: Fix build with Qt 5.9.
Jul 12 2019, 2:14 PM
wbauer committed R220:9acb42845378: Fix build with Qt 5.9 (authored by wbauer).
Fix build with Qt 5.9
Jul 12 2019, 2:14 PM
wbauer updated the test plan for D22415: Actually set minimum Qt version.
Jul 12 2019, 10:02 AM
wbauer updated the diff for D22412: Fix build with Qt 5.9.

Remove accidental trailing spaces.

Jul 12 2019, 9:36 AM
wbauer requested review of D22415: Actually set minimum Qt version.
Jul 12 2019, 9:31 AM
wbauer added a comment to D22412: Fix build with Qt 5.9.

Btw, on an unrelated note:
I just noticed that it doesn't actually specify a minimum Qt version, even though it seems to be the intention:

set(QT_MIN_VERSION 5.9.0)
find_package(Qt5 REQUIRED COMPONENTS
    Core DBus Gui Sql Widgets Xml
)
Jul 12 2019, 9:18 AM
wbauer requested review of D22412: Fix build with Qt 5.9.
Jul 12 2019, 9:17 AM

Jul 7 2019

wbauer added a comment to D22300: Use QTWEBENGINE_VERSION not QT_VERSION for WebEngine feature detection.

Nevermind though, lmontel fixed it meanwhile with commit 7b3249547b92.

Jul 7 2019, 9:03 AM · KDE PIM

Jul 6 2019

wbauer added a comment to D22300: Use QTWEBENGINE_VERSION not QT_VERSION for WebEngine feature detection.

This doesn't work as intended (at least here on openSUSE).
I just built it with QtWebEngine 5.13(.0), and it incorrectly took the < 5.13.0 branch.

Jul 6 2019, 10:29 PM · KDE PIM

Jul 4 2019

wbauer committed R201:c7c9a3a83d5d: Merge branch 'Applications/19.04' (authored by wbauer).
Merge branch 'Applications/19.04'
Jul 4 2019, 4:36 PM
wbauer committed R201:46adc9c9a6dc: [articleviewerwebengine] Reinstate crash "workaround" for Qt 5.13 (authored by wbauer).
[articleviewerwebengine] Reinstate crash "workaround" for Qt 5.13
Jul 4 2019, 4:36 PM
wbauer closed D22228: [articleviewerwebengine] Reinstate crash "workaround" for Qt 5.13.
Jul 4 2019, 4:36 PM · KDE PIM

Jul 3 2019

wbauer closed D21952: [ftp] Fix wrong access time in Ftp::ftpCopyGet().
Jul 3 2019, 8:19 PM · Frameworks
wbauer committed R241:f9c2dfb6e8df: [ftp] Fix wrong access time in Ftp::ftpCopyGet() (authored by wbauer).
[ftp] Fix wrong access time in Ftp::ftpCopyGet()
Jul 3 2019, 8:19 PM
wbauer updated subscribers of D21952: [ftp] Fix wrong access time in Ftp::ftpCopyGet().
Jul 3 2019, 7:34 PM · Frameworks
wbauer updated the test plan for D22228: [articleviewerwebengine] Reinstate crash "workaround" for Qt 5.13.
Jul 3 2019, 7:32 PM · KDE PIM
wbauer updated the test plan for D22228: [articleviewerwebengine] Reinstate crash "workaround" for Qt 5.13.
Jul 3 2019, 7:31 PM · KDE PIM