qgit icon indicating copy to clipboard operation
qgit copied to clipboard

Running latest qgit on Mac OS X 10.6.8 has memory leaks

Open NucleaPeon opened this issue 5 years ago • 6 comments

When running qgit on the command line on snow leopard, I get these messages:

qgit-mem-leak

Applying the CocoaInitializer fix results in no error messages:

prevent-mem-leaks

You can find the fixes in my forked repo: https://github.com/NucleaPeon/qgit

I built qgit using Qt 5.3.2.

Thanks for building/maintaining this piece of software.

NucleaPeon avatar Mar 05 '20 09:03 NucleaPeon

Hello Dan

Thank you very much for this contribution. It is very much appreciated, in particular given that I don't have access to a MacOS machine for development so your help is very useful.

Please consider making a pull request so that I can merge your changes in the main qgit repo.

Thanks Cristian

tibirna avatar Mar 09 '20 02:03 tibirna

I made a pull request, but this is the first time doing so on github. Let me know if I need to take additional steps. I squashed it into one commit and put it in a branch.

https://github.com/NucleaPeon/qgit/tree/macosx-10.6.8-memleak-fix

NucleaPeon avatar Mar 10 '20 01:03 NucleaPeon

I'd hold off merging it just yet, there's an issue with some of the main menu being duplicated.

NucleaPeon avatar Mar 12 '20 03:03 NucleaPeon

OK ready for merge

NucleaPeon avatar Mar 13 '20 17:03 NucleaPeon

Your pull request didn't work (I can't see it in my list). From what I can tell, you made against your repository.

When generating the pull request, choose the base repo to be tibirna/qgit.

tibirna avatar Mar 15 '20 00:03 tibirna

Sorry, I'm unfamiliar with github's workflow. I've forked directly from you and patched in the changes into their own branch. https://github.com/tibirna/qgit/pull/91

Thank you

NucleaPeon avatar Mar 15 '20 19:03 NucleaPeon