Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MrPhilTX
Copy link

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
Copy link

ghost commented Mar 28, 2022

CLA assistant check
All CLA requirements met.

@Neustradamus
Copy link

@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