Skip to content

Add GLib library, add GLib tests (some new bugs found)#231

Merged
danmar merged 1 commit into
cppcheck-opensource:masterfrom
scriptum:rpg-add-glib-tests
Feb 2, 2014
Merged

Add GLib library, add GLib tests (some new bugs found)#231
danmar merged 1 commit into
cppcheck-opensource:masterfrom
scriptum:rpg-add-glib-tests

Conversation

@scriptum

@scriptum scriptum commented Feb 1, 2014

Copy link
Copy Markdown
Contributor

I cannot submit issue because of logging in bug on SF, so I decided to write some test cases for GLib library. There already was some GLib tests but wasn't glib library itself. This patch will add library and corresponding library checks.

Note that malloc ang g_malloc behavior is very different by some reason (there are some bugs in analyzer while using GTK library). It doesn't break current tests but adds more todo-s.

@danmar

danmar commented Feb 2, 2014

Copy link
Copy Markdown
Collaborator

It would be good to use LOAD_LIB. The testmemleak already uses LOAD_LIB for posix.cfg.

@scriptum

scriptum commented Feb 2, 2014

Copy link
Copy Markdown
Contributor Author

Okay, thanks, I'll fix it. There are many bugs while analyzing using GLib (gtk.cfg is actually contains GLib functions), I'm preparing PR's with some fixes:)

Also gtk.cfg should be updated, e.g. GLib has no function g_strcmp, there is g_strcmp0 and so on. It's good to always duplicate all mallocs with g_malloc - some bugs in Library class could be discovered.

@scriptum

scriptum commented Feb 2, 2014

Copy link
Copy Markdown
Contributor Author

Rebased, added new function to gtk.cfg.

UPD. Seems that there is some sort of conflict with posix - I disabled posix lib but all tests pass anyway...

@scriptum

scriptum commented Feb 2, 2014

Copy link
Copy Markdown
Contributor Author

Now it shouldn't conflict.

danmar added a commit that referenced this pull request Feb 2, 2014
Add GLib library, add GLib tests (some new bugs found)
@danmar danmar merged commit ed4fa9f into cppcheck-opensource:master Feb 2, 2014
@danmar

danmar commented Feb 2, 2014

Copy link
Copy Markdown
Collaborator

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.

2 participants