Skip to content
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

Fixes for raster memory-leaks #666

Closed
wants to merge 6 commits into from
Closed

Conversation

jtorresfabra
Copy link

Please, dismiss the changes to FindGDAL.cmake.

Two leaks fixed:

  1. Missing delete in qgsrasterresamplefilter.cpp
  2. Using qDeleteAll() instead of a leaky method to destroy pointers in multibandcolorrenderer.cpp

jtorresfabra and others added 3 commits June 10, 2013 16:17
When working in a external application and doing make install in qgis it misses some raster headers.
Current FindGdal.cmake is not working well in CentOS
@ghost ghost assigned blazek Jun 19, 2013
Jordi Torres added 2 commits June 19, 2013 11:45
…ion. Without this fix memory increases till freeze the syste
@nirvn
Copy link
Contributor

nirvn commented Jun 20, 2013

@blazek ping :)

I'm seeing long QGIS freezes and once in a while a crash when dealing with large rasters (i.e. combined landsat-8 16-bit bands). Would love to see if the above commits improve the situation.

@NathanW2
Copy link
Member

Is this pull request still valid. If it is could you clean it up @jtorresfabra so that we can merge it. Remove any commits related to local cmake changes.

@jorditorres
Copy link

Hi @NathanW2,

Just done a new pull request from a new account, as my normal user branch is used right now in production, so cannot change this branch right now. Here you have the new pull request #772

You can close this one.

Thanks

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.

None yet

5 participants