Uninstall should remove all variables

Hi guys,

Thanks a lot for the great work and I have been seeing the changes in the version control (commits).

It seems several new settings have been introduced:

sharethis_publisherID

However it seems the uninstall function chosen_uninstall only deletes the variable sharethis_publisherID, in the sharethis.install file.

I have tried to make a patch, attached to this ticket, to try to resolve this issue: Remove_all_settings_variables_during_module_uninstall.patch

I would tend to think calls for deleting variables (variable_del) should probably be comitted together in the .install file when the .module or .inc files are modified to add variables (variable_get, usually in module's configuration settings form).

I would greatly appreciate if you could take some time to test/review the patch and give us your feedback on this feature request.

Any feedback, comments, testing is highly appreciated.
Thanks in advance.