TortoiseSVN Notification Sound "Problem"

33 views
Skip to first unread message

Achim Kalwa

unread,
May 13, 2021, 1:16:00 PM5/13/21
to TortoiseSVN
Just a small issue, but very annoying (for me):

Several actions in the client are notified with a small sound, which are all played with different audio volumes. Each notification gets its own volume control slider in the sound mixer (see screenshot). This is because in SVNProgressDlg.cpp all calls to PlaySound() do not use the SND_SYSTEM flag. If this flag would be specified, the sound is assigned to the audio session for system notification sounds.

My proposal is to use SND_SYSTEM either as the new default; or make it an advanced configure option. There is already a "PlaySound" option, which could be converted from Boolean to Integer and interpreted like this:
0 = Off (former "false")
1 = On  (former "true"), call PlaySound without SND_SYSTEM like current version
2 = On  (New), using the SND_SYSTEM flag.

Thanks for reading,

Achim

TSVN Commit Failed Sound.png
TSVN Update Finished Sound.png

Tony Rietwyk

unread,
May 13, 2021, 11:12:45 PM5/13/21
to TortoiseSVN
FWIW, I used to have directly connected speakers to the computer, and never had this type of problem.  When I bought new monitors, I changed to use the inbuilt USB speakers.  Now I have this same problem, especially for MS Teams.  I change the volume for the app, but stupid Windows doesn't remember them, and keeps setting it back to the device max.  

Achim Kalwa

unread,
May 15, 2021, 6:30:50 AM5/15/21
to TortoiseSVN
So what do you think; is this problem serious enough to report as a bug, or as a new feature request? I could not find any public issue tracker except for this forum.

Achim
Reply all
Reply to author
Forward
0 new messages