-
Notifications
You must be signed in to change notification settings - Fork 778
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
Memory Leak on XojFont #10
Labels
Comments
There is a macro "XOJ_INIT_TYPE(SaveJob);" missing e.g. in a copy constructor. I used this macros because its faster than use e.g. valgrind, and it's always on, also if the user don't mind about it... The disadvantage is: it has to be inserted in each constructor / destructor. |
Good call - it's in FontButton when it returns an XojFont without constructing a new one. Fixed... |
wbrenna
added a commit
that referenced
this issue
Aug 26, 2013
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If the font size (probably type as well) is changed, it will leak "-1" objects of type XojFont. This does not actually require typing anything in the new or old font.
The text was updated successfully, but these errors were encountered: