Skip to content

Added optional sound when leaks are detected; clean up of installer#17

Open
MrPhilTX wants to merge 4 commits into
Azure:masterfrom
MrPhilTX:master
Open

Added optional sound when leaks are detected; clean up of installer#17
MrPhilTX wants to merge 4 commits into
Azure:masterfrom
MrPhilTX:master

Conversation

@MrPhilTX

Copy link
Copy Markdown

The commit by grrava is already included in a (currently) open pull request via the avadae user. I needed their changes for my project.

My changes include:

  • a new vld.ini file option: AudioAlertOnLeak. If on, the system will beep if a leak is detected
  • the VLD directories are put in the VC Directories section of the Microsoft.Cpp.Win32.user.props file, as the documentation and my experiences suggest works.
  • the .iss code to install received some refactoring cleanup in the function I was working in

I wasn't sure how to handle the versioning, so I guessed. Apologies to avadae/grrava if I have overstepped by pulling around them.

grrava and others added 3 commits March 9, 2022 10:31
…orted in x86 builds. Converted projects to vs2022
…tup.iss: puts VLD directories in the VC Directories section of the .props
@ghost

ghost commented Mar 28, 2022

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@Neustradamus

Copy link
Copy Markdown

@MrPhilTX, @mattdurak: Any progress on it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants